test: only include WebSocket in WPT Report where it's landed by panva · Pull Request #2351 · nodejs/undici (original) (raw)
Conversation
This makes the inclusion of WebSocket in the daily WPT Report uploaded depend on whether it has landed in the respective tested release line or not (or whether it's already made global by default).
Currently it's reported even for releases where the experimental flag is not available (yet).
Codecov Report
✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.78%. Comparing base (e39a632) to head (a237f46).
⚠️ Report is 1357 commits behind head on main.
Additional details and impacted files
@@ Coverage Diff @@ ## main #2351 +/- ##
Coverage 85.54% 85.78% +0.23%
Files 76 76
Lines 6858 6871 +13- Hits 5867 5894 +27
- Misses 991 977 -14
☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.
🚀 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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
kodiakhq Bot referenced this pull request in X-oss-byte/Canary-nextjs
kodiakhq Bot referenced this pull request in X-oss-byte/Nextjs
This was referenced
Nov 2, 2023
kodiakhq Bot referenced this pull request in ascorbic/unpic-img
renovate Bot referenced this pull request in specfy/specfy
gr2m referenced this pull request in octokit/rest.js
crysmags pushed a commit to crysmags/undici that referenced this pull request
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
[ Show hidden characters]({{ revealButtonHref }})