FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

dgram: mark socket* methods end-of-life by anonrig · Pull Request #58520 · nodejs/node · GitHub

/ node Public

dgram: mark socket* methods end-of-life - #58520

Closed
anonrig wants to merge 1 commit into
nodejs:mainfrom
anonrig:yagiz/eol-dgram
Closed

dgram: mark socket* methods end-of-life#58520
anonrig wants to merge 1 commit into
nodejs:mainfrom
anonrig:yagiz/eol-dgram

Conversation

anonrig commented May 31, 2025

Copy link
Copy Markdown
Member

Since runtime deprecating this, it has been 13 major versions...

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/net

nodejs-github-bot added dgram Issues and PRs related to the dgram subsystem / UDP. needs-ci PRs that need a full CI run. labels May 31, 2025
anonrig force-pushed the yagiz/eol-dgram branch from b12f3c9 to 9e2c782 Compare May 31, 2025 00:17
anonrig added semver-major PRs that contain breaking changes and should be released in the next major version. deprecations Issues and PRs related to deprecations. labels May 31, 2025
anonrig force-pushed the yagiz/eol-dgram branch from 9e2c782 to 4fb3225 Compare May 31, 2025 00:19

jasnell commented May 31, 2025

Copy link
Copy Markdown
Member

Duplicate ;-) #58474

anonrig commented May 31, 2025

Copy link
Copy Markdown
Member Author

Dammit.

anonrig closed this May 31, 2025
anonrig deleted the yagiz/eol-dgram branch May 31, 2025 00:20

anonrig commented May 31, 2025

Copy link
Copy Markdown
Member Author

Duplicate ;-) #58474

What's funny is that, I approved that PR...

codecov Bot commented May 31, 2025
edited
Loading

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.23%. Comparing base (9524908) to head (4fb3225).
Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #58520      +/-   ##
==========================================
- Coverage   90.25%   90.23%   -0.03%     
==========================================
  Files         635      635              
  Lines      187635   187564      -71     
  Branches    36861    36853       -8     
==========================================
- Hits       169347   169243     -104     
- Misses      11055    11099      +44     
+ Partials     7233     7222      -11     
Files with missing lines Coverage Δ
lib/dgram.js 97.45% <100.00%> (-0.17%) ⬇️

... and 30 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.

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deprecations Issues and PRs related to deprecations. dgram Issues and PRs related to the dgram subsystem / UDP. needs-ci PRs that need a full CI run. semver-major PRs that contain breaking changes and should be released in the next major version.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL