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

chore: bootstrap Poetry in CI via pipx #1372

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ahal
Copy link
Contributor

@ahal ahal commented Nov 7, 2023

The poetry installation docs state very clearly that you can't install Poetry into the same Python environment as the one it is management. The old method of pip install -r poetry.txt was doing exactly that.

Instead, let's use pipx which hides Poetry away in an isolated environment. While this doesn't check hashes, it's likely not a big deal for this case.

As far as reproducing locally, I recommend also using pipx. It's even possible to use pipx to install multiple versions of Poetry:

$ pipx install --suffix shipit poetry==<version>
$ poetry@shipit install

This way, if you really need the exact same version that Shipit CI uses, it's possible.

The poetry installation docs state very clearly that you can't install
Poetry into the same Python environment as the one it is management. The
old method of `pip install -r poetry.txt` was doing exactly that.

Instead, let's use `pipx` which hides Poetry away in an isolated
environment. While this doesn't check hashes, it's likely not a big
deal for this case.

As far as reproducing locally, I recommend also using `pipx`. It's even
possible to use `pipx` to install multiple versions of Poetry:

  $ pipx install --suffix shipit poetry==<version>
  $ poetry@shipit install

This way, if you *really* need the exact same version that Shipit CI
uses, it's possible.
@ahal
Copy link
Contributor Author

ahal commented Nov 13, 2023

Not sure what's going on here, but I'm unlikely to find time to dig into it for awhile.

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

1 participant