-
Notifications
You must be signed in to change notification settings - Fork 21
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
github: add --ignore-installed
to pip install
of requirements
#293
Conversation
25343b4
to
151b639
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.
Looks OK but can you explain what's going on?
The rabbit master hopped to this PR too quickly! I'm fairly certain it's a problem with a mismatch between system installed packages in the container and what pip thinks is installed. As to why it's showing up now, I have no idea. 🤔 |
I guess I'll fess up to finding the "solution" on a very random blog |
Maybe use commit title: "github: add Also the workflow specifies |
I'm fine with that, but FWIW, the error message that prompted this specifies debian:
|
Problem: recently, our github workflow for spelling and linkcheck got confused between pre-installed ubuntu packages and python's pip installed packages. Just ignore the ubuntu packages. Install our own. It doesn't seem to take much longer.
151b639
to
80a8adb
Compare
--ignore-installed
to pip install
of requirements
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.
👍
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!
Problem: recently, our github workflow for spelling and
linkcheck got confused between pre-installed debian packages
and python's pip installed packages.
Just ignore the debian packages. Install our own. It doesn't
seem to take much longer.
From: https://pip.pypa.io/en/stable/cli/pip_install/