-
Notifications
You must be signed in to change notification settings - Fork 999
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
[22.01] Backport and fix webapps tests #13478
[22.01] Backport and fix webapps tests #13478
Conversation
Also: - Remove redundant `galaxy.schema` module from the `app` package (it's already in `data`) - Fix requirements of the `app` and `webapps` packages - Add missing `PyJWT` and `python-dateutil` to `tool.poetry.dependencies` (which were already included in the pinned requirements as deps of deps)
And restore unit tests to proper place.
847bb4e
to
873d43e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
as far as I can tell, this should work. Thanks!
@@ -1,4 +1,3 @@ | |||
mock-ssh-server | |||
pykwalify |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just noticed while merging forward that this shouldn't have been moved to packages/app/requirements.txt
, so I haven't retained this change in the merge. Probably not that important to fix in 22.01.
This broke the packing tests on dev after merging #13461 into dev,
Backport of #13402
How to test the changes?
(Select all options that apply)
License