-
Notifications
You must be signed in to change notification settings - Fork 13k
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
Stable backport of #73613 #74746
Stable backport of #73613 #74746
Conversation
Thus we avoid propagation of a local the moment we encounter references to it.
@bors r+ p=1000 rollup=never Thanks! I'll invalidate dev-static once this lands so we can get artifacts for testing. |
📌 Commit 32166ab has been approved by |
⌛ Testing commit 32166ab with merge 9e04db83e57fdce346a7c31587b760e70fe1a538... |
Your PR failed (pretty log, raw log). Through arcane magic we have determined that the following fragments from the build log may contain information about the problem. Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
💔 Test failed - checks-actions |
Well, that's not good.
cc @mati865 |
I won't have access to any desktop until tomorrow. I think this doesn't happen on nightly right? |
The PR #73188 |
@Mark-Simulacrum I have checked and #73188 is not present on stable. Do you want to backport it or should I search for band-aid workaround? |
I'm fine with backporting it, will do so sometime today. Thanks! |
@bors r+ p=1000 rollup=never |
📌 Commit 41895ca has been approved by |
☀️ Test successful - checks-actions, checks-azure |
This is the backport of #73613 to stable.
r? @ghost
cc @Mark-Simulacrum
In addition the tests added in the original PR passing, I've also confirmed that the test case in #74739 works correctly.