-
-
Notifications
You must be signed in to change notification settings - Fork 33.8k
http: move writeHeader to end-of-life #60635
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
base: main
Are you sure you want to change the base?
Conversation
|
Review requested:
|
2c32975 to
c798da4
Compare
|
cc: @nodejs/tsc |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #60635 +/- ##
==========================================
- Coverage 88.54% 88.54% -0.01%
==========================================
Files 703 703
Lines 208077 208072 -5
Branches 40082 40089 +7
==========================================
- Hits 184252 184246 -6
+ Misses 15848 15845 -3
- Partials 7977 7981 +4
🚀 New features to boost your workflow:
|
AugustinMauroy
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGMT !
mcollina
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
|
There is a problem with the docs linter I think. Can you fix? |
c798da4 to
ff0e79d
Compare
|
Yes, it was already fixed in #60636, I just did a rebase |
It's been over 7 years since it was deprecated in documentation only, and it’s been runtime-deprecated since version 25. I think it can finally be moved to end-of-life for version 26, since its usage is practically nonexistent.