Skip to content

Conversation

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-botnodejs-github-bot commented Mar 8, 2025

This is an automated update of the WPT for url to web-platform-tests/wpt@d2c3936.

@nodejs-github-botnodejs-github-bot added the test Issues and PRs related to the tests. label Mar 8, 2025
@nodejs-github-bot
Copy link
CollaboratorAuthor

Review requested:

  • @nodejs/web-standards

@nodejs-github-botnodejs-github-bot added the needs-ci PRs that need a full CI run. label Mar 8, 2025
@nodejs-github-botnodejs-github-bot changed the title test: update WPT for urlpattern to ae2959e731test: update WPT for url to d2c3936436Mar 8, 2025
@codecov
Copy link

codecovbot commented Mar 8, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.23%. Comparing base (3a497dc) to head (54cf483).
Report is 4 commits behind head on main.

Additional details and impacted files
@@ Coverage Diff @@## main #57368 +/- ## ========================================== + Coverage 90.22% 90.23% +0.01%  ========================================== Files 630 630 Lines 185259 185259 Branches 36245 36249 +4 ========================================== + Hits 167143 167164 +21 + Misses 11063 11058 -5 + Partials 7053 7037 -16 

see 24 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.

@targos
Copy link
Member

There seems to be a conflict with the update workflow. This PR was changed from an update of urlpattern to an update of url tests.

@anonrig
Copy link
Member

This PR also needs to be updated. I'll just close this and retrigger it.

@anonriganonrig closed this Mar 8, 2025
@anonriganonrig deleted the actions/update-wpt-urlpattern branch March 8, 2025 18:07
@JonasBa
Copy link
Contributor

@targos I think I found the root cause here, would you mind reviewing this PR?

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

5 participants

@nodejs-github-bot@targos@anonrig@JonasBa