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

fix(strict-deps): fail if venv is inconsistent #1784

Merged
merged 4 commits into from
Aug 6, 2024
Merged

Conversation

lengau
Copy link
Collaborator

@lengau lengau commented Aug 2, 2024

This runs pip check to ensure that all the packages in the charm virtualenv have all of their required dependencies, failing the build if they do not.

Fixes #1781

This runs `pip check` to ensure that all the packages in the charm
virtualenv have all of their required dependencies, failing the build
if they do not.

Fixes #1781
@lengau lengau force-pushed the work/1781/pip-check branch from ab8179d to ba92c13 Compare August 2, 2024 20:05
lengau added 3 commits August 2, 2024 18:22
This adds a test for when strict-dependencies is used and packages are
missing from the requirements file.
@lengau lengau marked this pull request as ready for review August 2, 2024 22:39
@lengau lengau requested a review from a team August 2, 2024 22:39
charmcraft/charm_builder.py Show resolved Hide resolved
@lengau lengau requested a review from a team August 5, 2024 18:56
@lengau lengau enabled auto-merge August 5, 2024 19:32
@lengau lengau added this pull request to the merge queue Aug 6, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 6, 2024
@lengau lengau added this pull request to the merge queue Aug 6, 2024
Merged via the queue into main with commit fd21df2 Aug 6, 2024
17 checks passed
@lengau lengau deleted the work/1781/pip-check branch August 6, 2024 19:05
lengau added a commit that referenced this pull request Aug 6, 2024
This runs `pip check` to ensure that all the packages in the charm
virtualenv have all of their required dependencies, failing the build if
they do not.

Fixes #1781
lengau added a commit that referenced this pull request Aug 6, 2024
This runs `pip check` to ensure that all the packages in the charm
virtualenv have all of their required dependencies, failing the build if
they do not.

Fixes #1781
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.

charmcraft 3.1.1 "successfully" packs charm, but python packages are missing in the venv
3 participants