-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
misc. improvements to group filtering logic and related options #5338
Merged
Commits on Mar 24, 2022
-
Configuration menu - View commit details
-
Copy full SHA for a5c1479 - Browse repository at this point
Copy the full SHA a5c1479View commit details -
Configuration menu - View commit details
-
Copy full SHA for 4b0066f - Browse repository at this point
Copy the full SHA 4b0066fView commit details -
refactor group command logic and simplify installer
This change removed code duplication for group option handling in commands and group activation in installer. The installer now only allows for `only_groups` and removes `with_groups`/`without_groups`. Additionally, this change enables code reuse for group flags in the update command and centralises deprecated option handling logic for these options. Further, this fixes regression when using `update --no-dev`.
Configuration menu - View commit details
-
Copy full SHA for d272663 - Browse repository at this point
Copy the full SHA d272663View commit details -
Configuration menu - View commit details
-
Copy full SHA for 01774e1 - Browse repository at this point
Copy the full SHA 01774e1View commit details -
groups: warn when with/without is used with only
This change adds a warning that values of `with`/`without` options are ignored when used with `only`.
Configuration menu - View commit details
-
Copy full SHA for f338da9 - Browse repository at this point
Copy the full SHA f338da9View commit details -
Configuration menu - View commit details
-
Copy full SHA for 77f1d95 - Browse repository at this point
Copy the full SHA 77f1d95View commit details -
tests: fix and improve installer test coverage
This change improves test scenarios for installer with dependency groups. Previously, the usage of populated installed repositories could lead to tests incorrectly passing as no operations were performed. These cases now forces syncronisation and uses a clean installed repository instance instead.
Configuration menu - View commit details
-
Copy full SHA for 36c0032 - Browse repository at this point
Copy the full SHA 36c0032View commit details -
Configuration menu - View commit details
-
Copy full SHA for c71cdfc - Browse repository at this point
Copy the full SHA c71cdfcView commit details -
Configuration menu - View commit details
-
Copy full SHA for 442ff8e - Browse repository at this point
Copy the full SHA 442ff8eView commit details -
tests: improve coverage for install group options
Co-authored-by: Randy Döring <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 1a053c9 - Browse repository at this point
Copy the full SHA 1a053c9View commit details -
Configuration menu - View commit details
-
Copy full SHA for 16cc701 - Browse repository at this point
Copy the full SHA 16cc701View commit details -
Configuration menu - View commit details
-
Copy full SHA for f6bcfb9 - Browse repository at this point
Copy the full SHA f6bcfb9View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.