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

Add superseding flag logic #1078

Merged
merged 3 commits into from
May 31, 2024
Merged

Add superseding flag logic #1078

merged 3 commits into from
May 31, 2024

Conversation

elliotgunton
Copy link
Collaborator

@elliotgunton elliotgunton commented May 31, 2024

This PR adds a superseding relationship between flags, specified by the _SUPERSEDING_FLAGS dictionary in _global_config.py

Also refactored and removed unused code

Signed-off-by: Elliot Gunton <elliotgunton@gmail.com>
Signed-off-by: Elliot Gunton <elliotgunton@gmail.com>
Signed-off-by: Elliot Gunton <elliotgunton@gmail.com>
@elliotgunton elliotgunton added the type:skip-changelog A change that does not require a changelog entry label May 31, 2024
@elliotgunton elliotgunton added type:task A general task semver:minor A change requiring a minor version bump and removed type:skip-changelog A change that does not require a changelog entry labels May 31, 2024
@elliotgunton elliotgunton mentioned this pull request May 31, 2024
4 tasks
@elliotgunton elliotgunton enabled auto-merge (squash) May 31, 2024 09:36
@elliotgunton elliotgunton changed the title Refactor flag logic Add superseding flag logic May 31, 2024
@elliotgunton elliotgunton enabled auto-merge (squash) May 31, 2024 09:36
@elliotgunton elliotgunton merged commit fd79fbf into main May 31, 2024
26 checks passed
@elliotgunton elliotgunton deleted the refactor-flags branch May 31, 2024 20:27
elliotgunton added a commit that referenced this pull request Sep 13, 2024
* Setting script_pydantic_io flag also sets the `script_annotations`
flag

Follow up from #1078, docs still mentioned users have to set the
superseded flag

Signed-off-by: Elliot Gunton <elliotgunton@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver:minor A change requiring a minor version bump type:task A general task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants