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

feat(plugins/poetry): add poetry and pip extra args #940

Merged
merged 4 commits into from
Dec 16, 2024
Merged

Conversation

lengau
Copy link
Contributor

@lengau lengau commented Dec 16, 2024

This allows more customisation of the poetry and pip commands for advanced users.

  • Have you signed the CLA?

lengau added a commit to canonical/charmcraft that referenced this pull request Dec 16, 2024
This moves the environment variable configuration to use a parameter.
This is needed because pip does not load environment variables in a
deterministic order, but the order of these parameters matters.

Requires canonical/craft-parts#940

Fixes #2025

CRAFT-3794
This allows more customisation of the poetry and pip commands for
advanced users.
@lengau lengau force-pushed the work/932/CRAFT-3792 branch from 7b70112 to 7fa93c2 Compare December 16, 2024 18:00
@lengau lengau marked this pull request as ready for review December 16, 2024 20:20
Copy link
Contributor

@medubelko medubelko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One-word suggestion.

docs/reference/changelog.rst Outdated Show resolved Hide resolved
@lengau lengau merged commit e55d6bc into main Dec 16, 2024
13 checks passed
@lengau lengau deleted the work/932/CRAFT-3792 branch December 16, 2024 22:53
lengau added a commit to canonical/charmcraft that referenced this pull request Dec 17, 2024
This moves the environment variable configuration to use a parameter.
This is needed because pip does not load environment variables in a
deterministic order, but the order of these parameters matters.

Requires canonical/craft-parts#940

Fixes #2025

CRAFT-3794
lengau added a commit to tmihoc/charmcraft that referenced this pull request Dec 19, 2024
This moves the environment variable configuration to use a parameter.
This is needed because pip does not load environment variables in a
deterministic order, but the order of these parameters matters.

Requires canonical/craft-parts#940

Fixes canonical#2025

CRAFT-3794
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.

Add poetry-export-extra-args and poetry-pip-extra-args options to the poetry plugin
4 participants