Repository navigation
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the Unlinked AccountsThe following contributors have not linked their GitHub and WordPress.org accounts: @Copilot. Contributors, please read how to link your accounts to ensure your work is properly credited in WordPress releases. If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
🤖 PR meta 🤖📦 Bundle sizeSize Change: 0 B Total Size: 7.75 MB
🏁 Flaky testsShow the failuresSome tests passed with failed attempts. The failures may not be related to this commit but are still reported for visibility. See the documentation for more information. should paste block styles using keyboard in
|
74cebda to
b18f4ca
Compare
b18f4ca to
c31c0ef
Compare
c31c0ef to
cb67e1c
Compare
cb67e1c to
4d8f9ec
Compare
Part of #82781.
What?
Makes the "should update the HTML only for the latest navigation" test in
test/e2e/specs/interactivity/router-navigate.spec.tswait until the browser has requested both linked pages before letting either request through.Why?
The test holds each page request in a
page.route()handler, and the functions that let a request through (resolveLink1andresolveLink2) are only assigned once those handlers run. The test only waited for the page's own navigation counter, which can reach 2 before Playwright has run the handlers, so callingresolveLink2()could fail withTypeError: resolveLink2 is not a function. It hasn't been seen failing on this stack, but this branch has the same test, so the fix is included for consistency.Upstream
A straight cherry-pick of
e693c18(#80178) fromtrunk.Use of AI Tools
This PR was created by Claude Code under my supervision. All code should be treated as AI-produced and not yet reviewed by a human until this PR is marked Ready for Review.
🤖 Generated with Claude Code