Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

remove stale dependencies on state mv #23791

Merged
merged 1 commit into from
Jan 6, 2020
Merged

remove stale dependencies on state mv #23791

merged 1 commit into from
Jan 6, 2020

Conversation

jbardin
Copy link
Member

@jbardin jbardin commented Jan 6, 2020

Clear any Dependencies if there is an entry matching a state mv from
address. While stale dependencies won't directly effect any current
operations, clearing the list will allow them to be recreated in their
entirety during refresh. This will help future releases that may rely
solely on the pre-calculated dependencies for destruction ordering.

Closes #23711

@jbardin jbardin requested a review from a team January 6, 2020 19:04
@jbardin jbardin force-pushed the jbardin/state-mv branch 2 times, most recently from 918b26e to 5d1e3f8 Compare January 6, 2020 19:32
Clear any Dependencies if there is an entry matching a `state mv` from
address. While stale dependencies won't directly effect any current
operations, clearing the list will allow them to be recreated in their
entirety during refresh. This will help future releases that may rely
solely on the pre-calculated dependencies for destruction ordering.
@jbardin jbardin merged commit ae40706 into master Jan 6, 2020
@jbardin jbardin deleted the jbardin/state-mv branch January 6, 2020 21:02
@apparentlymart apparentlymart added this to the 0.12.19 milestone Jan 7, 2020
@ghost
Copy link

ghost commented Feb 6, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators Feb 6, 2020
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

state mv doesn't remove stale dependencies
3 participants