-
Notifications
You must be signed in to change notification settings - Fork 1
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
Make using dependencies branch toggleable #184
Conversation
Update name for this input in workflow for updating pyproject.toml to be `target_branch`, deprecating `permanent_dependencies_branch`.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #184 +/- ##
=======================================
Coverage 81.11% 81.11%
=======================================
Files 12 12
Lines 900 900
=======================================
Hits 730 730
Misses 170 170 ☔ View full report in Codecov by Sentry. |
This reverts commit ecff44c.
The new warning message has been tested (see this run) and looks to work as intended. |
Handle all the different cases between the now deprecated 'permanent_dependencies_branch' input value and the replacement 'target_branch' value with more appropriate warning messages and results.
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.
Some of the comments/docs could be synchronized a bit more regarding permanent_dependencies_branch
Update name for this input in workflow for updating pyproject.toml to be
target_branch
, deprecatingpermanent_dependencies_branch
.Closes #183
This will make it easier for users to choose or indeed not choose to update dependencies in this way.