-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
[CT-470] [Feature] Add option for --defer to favor using --state node even if node exists in current target #5016
Comments
PR here of the proposed changes |
Hi @daniel-murray, thanks so much for the contribution! I'll give this PR a review shortly, but in the meantime can you please fill out our CLA so we can get this merged in? |
Hi @iknox-fa no problem at all! Thank you very much. We're waiting on our legal team here to review the CLA and unfortunately, they've asked us to wait until next week. I'll update here as soon as they do. |
Just adding a big plus one to getting this in! Any update @daniel-murray ? |
This issue has been marked as Stale because it has been open for 180 days with no activity. If you would like the issue to remain open, please remove the stale label or comment on the issue, or it will be closed in 7 days. |
Still want this! |
Is there an existing feature request for this?
Describe the Feature
This is the current behaviour of the
--defer
flag:The first criterion is always valid. However, it would be useful if the second criterion was optional.
This is because we re-use the same development schemas for multiple features. Therefore you could accidentally use an old version of a node you created for a different feature while using the --defer flag.
Describe alternatives you've considered
Drop all tables in our development schemas after each feature is complete.
It's not practical to delete all tables after each feature due to ongoing QA for old features.
Making
--favor-state
a completely different feature to--defer
The code change is minimal to implement this feature if used in conjunction with the code already created for
--defer
.Who will this benefit?
Anyone using the
--defer
flag so they can have more control over its behaviour.Are you interested in contributing this feature?
Yes would love to! We have a PR ready and tested.
Anything else?
No response
The text was updated successfully, but these errors were encountered: