Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Admin/update pip on dependency check #5827

Merged

Conversation

bblommers
Copy link
Collaborator

Simplify dependency test script.

New way to invoke the script:
scripts/dependency_test.sh acm

The CI is now responsible for retrieving the list of services, and invoking the script once per service. This ensures that we our virtual environments are properly delineated.

@bblommers bblommers added this to the 4.1 milestone Jan 7, 2023
@codecov-commenter
Copy link

Codecov Report

Merging #5827 (faa403c) into master (27a2e42) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #5827   +/-   ##
=======================================
  Coverage   96.18%   96.18%           
=======================================
  Files         753      753           
  Lines       73969    73969           
=======================================
  Hits        71150    71150           
  Misses       2819     2819           
Flag Coverage Δ
servertests 36.95% <ø> (ø)
unittests 96.12% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@bblommers bblommers merged commit 4165aa8 into getmoto:master Jan 7, 2023
@bblommers bblommers deleted the admin/update-pip-on-dependency-check branch January 7, 2023 22:54
@github-actions
Copy link
Contributor

github-actions bot commented Jan 8, 2023

This is now part of moto >= 4.1.0.dev6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants