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

[BUGFIX lts] Backport {{hash}} laziness #19543

Closed
wants to merge 1 commit into from
Closed

Conversation

pzuraq
Copy link
Contributor

@pzuraq pzuraq commented May 12, 2021

Backports the fixes from #19511

The internal _query property of LinkTo had to be made into an autotracking
dependency using dependentKeyCompat in order to handle the fact
that hash never updates now. This was not an issue on master since
LinkTo has been refactored there.

Backports the fixes from #19511

The internal `_query` property of LinkTo had to be made into an autotracking
dependency using dependentKeyCompat in order to handle the fact
that hash never updates now. This was not an issue on master since
LinkTo has been refactored there.
@rwjblue
Copy link
Member

rwjblue commented May 27, 2021

@pzuraq - Where are we at with this?

@pzuraq
Copy link
Contributor Author

pzuraq commented May 27, 2021

The PR needs to be updated to add the additional fixes that landed recently. I was planning on updating it once we landed those in release and were pretty sure it wouldn't need any more changes.

@chriskrycho
Copy link
Contributor

Status check here—IIRC, @pzuraq @rwjblue we ended up deciding this shouldn’t be back ported, because of the various issues that showed up? And in any case, the period in which we would have back ported it to 3.24 is loooong past. I think we should go ahead and close this.

@kategengler
Copy link
Member

At this point, we're clearly not backporting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

4 participants