-
-
Notifications
You must be signed in to change notification settings - Fork 401
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
CI Linux: Fixes for tests of optional packages #36708
Conversation
9a13f5b
to
5d7fae1
Compare
5d7fae1
to
4cf77b2
Compare
…thon packages declared as sagelib's install-requires
… packages starting with _
…s sagelib's install-requires
4cf77b2
to
9a82e56
Compare
Documentation preview for this PR (built with commit 9a82e56; changes) is ready! 🎉 |
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.
LGTM. Tests running locally with tox pass.
Thanks! Setting to blocker so that the corrected dependencies take effect in the CI. |
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.
Thanks for fixing this! I just noticed a cosmetic issue: sympy
is doubled in the dependencies_check
files.
Fixes test failure of
database_knotinfo
seen in GH Actions, as reported by @soehms in#36668 (comment)
Fixes same also for
database_cubic_hecke
andmathics
.To test:
FROM_DOCKER_TARGET=with-targets-pre TARGETS_PRE=config.status tox -e docker-ubuntu-bionic-gcc_8-maximal-incremental -- database_knotinfo database_cubic_hecke mathics
We also remove packages starting with
_
from the list of tested packages. The dependencies of_develop
pulled in the dummy packagegithub_cli
.We also use output groups to fold away some repetitive output.
📝 Checklist
⌛ Dependencies