Skip to content
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

quic: fix debug log #57689

Merged
merged 1 commit into from
Apr 3, 2025
Merged

Conversation

jakecastelli
Copy link
Member

No description provided.

@jakecastelli jakecastelli added the fast-track PRs that do not need to wait for 48 hours to land. label Mar 30, 2025

This comment was marked as outdated.

@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. quic Issues and PRs related to the QUIC implementation / HTTP/3. labels Mar 30, 2025
@jakecastelli jakecastelli force-pushed the fix-quic-debug-typos branch from c7737b2 to 5eda7aa Compare March 30, 2025 11:01
Copy link

codecov bot commented Mar 30, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.22%. Comparing base (af75d04) to head (5eda7aa).
Report is 21 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #57689      +/-   ##
==========================================
- Coverage   90.23%   90.22%   -0.01%     
==========================================
  Files         630      630              
  Lines      185055   185055              
  Branches    36221    36214       -7     
==========================================
- Hits       166984   166973      -11     
- Misses      11043    11046       +3     
- Partials     7028     7036       +8     

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

@cjihrig
Copy link
Contributor

cjihrig commented Mar 30, 2025

I'm not sure this can currently be fast tracked because it requires a full CI run and the CI is locked down.

@jakecastelli
Copy link
Member Author

oh yes, thanks for the reminder, forgot about the CI situation when I created the PR

@jakecastelli jakecastelli removed the fast-track PRs that do not need to wait for 48 hours to land. label Mar 30, 2025
@bjohansebas bjohansebas added the request-ci Add this label to start a Jenkins CI on a PR. label Apr 1, 2025
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Apr 1, 2025
@nodejs-github-bot
Copy link
Collaborator

@jasnell
Copy link
Member

jasnell commented Apr 2, 2025

I'm not sure this can currently be fast tracked because it requires a full CI run and the CI is locked down.

Well, that and quic is no longer built by default in CI ;-)

@jakecastelli jakecastelli added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. labels Apr 3, 2025
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Apr 3, 2025
@nodejs-github-bot nodejs-github-bot merged commit 10cff7b into nodejs:main Apr 3, 2025
63 checks passed
@nodejs-github-bot
Copy link
Collaborator

Landed in 10cff7b

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. needs-ci PRs that need a full CI run. quic Issues and PRs related to the QUIC implementation / HTTP/3.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants