Skip to content

test: update WPT for urlpattern to 3ffda23e5a - #58537

Closed
nodejs-github-bot wants to merge 2 commits into
mainfrom
actions/update-urlpattern-wpt
Closed

test: update WPT for urlpattern to 3ffda23e5a#58537
nodejs-github-bot wants to merge 2 commits into
mainfrom
actions/update-urlpattern-wpt

Conversation

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

This is an automated update of the WPT for urlpattern to web-platform-tests/wpt@3ffda23.

@nodejs-github-botnodejs-github-bot added the test Issues and PRs related to the tests. label Jun 1, 2025
@nodejs-github-bot

Copy link
Copy Markdown
CollaboratorAuthor

Review requested:

  • @nodejs/web-standards

@nodejs-github-botnodejs-github-bot added the needs-ci PRs that need a full CI run. label Jun 1, 2025
@codecov

codecovBot commented Jun 1, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.14%. Comparing base (f497881) to head (a5e3f8c).
Report is 291 commits behind head on main.

Additional details and impacted files
@@ Coverage Diff @@## main #58537 +/- ##
==========================================
- Coverage 90.23% 90.14% -0.09% 
==========================================
Files 635 636 +1 Lines 187580 188028 +448 Branches 36860 36889 +29 ==========================================
+ Hits 169265 169506 +241 - Misses 11101 11280 +179 - Partials 7214 7242 +28 

see 74 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@anonrig

Copy link
Copy Markdown
Member

Tentative tests shouldn't run since they are experimental. We should avoid running all of those things.

@targos
targos marked this pull request as draft June 7, 2025 09:10
@panva

panva commented Jun 10, 2025

Copy link
Copy Markdown
Member

Running tentative tests is still valuable if the spec is set in stone but WPTs not yet updated to reflect non-tentativeness.

If you don't wish to run these particular ones individually skip them or acknowledge their failure.

@panva
panva marked this pull request as ready for review June 10, 2025 13:24
@panvapanva added request-ci Add this label to start a Jenkins CI on a PR. and removed request-ci Add this label to start a Jenkins CI on a PR. labels Jun 10, 2025
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jun 10, 2025
@nodejs-github-bot

Copy link
Copy Markdown
CollaboratorAuthor

@nodejs-github-bot

Copy link
Copy Markdown
CollaboratorAuthor

@nodejs-github-bot

Copy link
Copy Markdown
CollaboratorAuthor

@panvapanva added the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 11, 2025
@nodejs-github-botnodejs-github-bot added commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Jun 11, 2025
@nodejs-github-bot

Copy link
Copy Markdown
CollaboratorAuthor
Commit Queue failed
- Loading data for nodejs/node/pull/58537
✔ Done loading data for nodejs/node/pull/58537
----------------------------------- PR info ------------------------------------
Title test: update WPT for urlpattern to 3ffda23e5a (#58537)
⚠ Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch nodejs-github-bot:actions/update-urlpattern-wpt -> nodejs:main
Labels test, needs-ci
Commits 2
- test: update WPT for urlpattern to 3ffda23e5a
- fixup! test: update WPT for urlpattern to 3ffda23e5a
Committers 2
- Node.js GitHub Bot <github-bot@iojs.org>
- Filip Skokan <panva.ip@gmail.com>
PR-URL: https://github.com/nodejs/node/pull/58537
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
Reviewed-By: Ethan Arrowood <ethan@arrowood.dev>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/58537
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
Reviewed-By: Ethan Arrowood <ethan@arrowood.dev>
--------------------------------------------------------------------------------
ℹ This PR was created on Sun, 01 Jun 2025 00:24:00 GMT
✔ Approvals: 3
✔ - James M Snell (@jasnell) (TSC): https://github.com/nodejs/node/pull/58537#pullrequestreview-2884856289
✔ - Filip Skokan (@panva) (TSC): https://github.com/nodejs/node/pull/58537#pullrequestreview-2913584556
✔ - Ethan Arrowood (@Ethan-Arrowood): https://github.com/nodejs/node/pull/58537#pullrequestreview-2913642970
✘ Last GitHub CI failed
ℹ Last Full PR CI on 2025-06-10T19:19:18Z: https://ci.nodejs.org/job/node-test-pull-request/67391/
- Querying data for job/node-test-pull-request/67391/
✔ Last Jenkins CI successful
--------------------------------------------------------------------------------
✔ Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/15581852088

panva pushed a commit that referenced this pull request Jun 11, 2025
PR-URL: #58537
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
Reviewed-By: Ethan Arrowood <ethan@arrowood.dev>
@panva

Copy link
Copy Markdown
Member

Landed in e027da9

@panvapanva closed this Jun 11, 2025
@panva
panva deleted the actions/update-urlpattern-wpt branch June 11, 2025 11:23
targos pushed a commit that referenced this pull request Jun 16, 2025
PR-URL: #58537
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
Reviewed-By: Ethan Arrowood <ethan@arrowood.dev>
@aduh95aduh95 added dont-land-on-v20.x dont-land-on-v22.x PRs that should not land on the v22.x-staging branch and should not be released in v22.x. and removed commit-queue-failed An error occurred while landing this pull request using GitHub Actions. labels Jul 21, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dont-land-on-v22.xPRs that should not land on the v22.x-staging branch and should not be released in v22.x.needs-ciPRs that need a full CI run.testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@nodejs-github-bot@anonrig@panva@jasnell@Ethan-Arrowood@bjohansebas@aduh95