| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
|
Review requested:
|
Sorry, something went wrong.
|
No, we need to keep the language about it being removed in a future version. |
Sorry, something went wrong.
Why? |
Sorry, something went wrong.
|
Because it's stronger. It means you must refactor, it's not just a recommendation. They should know that this is coming. |
Sorry, something went wrong.
|
I agree that the existing verbiage is better than the proposal because we're pretty darn sure upgrading is required (whereas the proposal makes it sound like merely a better option). |
Sorry, something went wrong.
|
Well we plan to remove it, but we can't know for sure if we are definitely going to remove it.
Surely you don't mean that, the current warning is not correct English. |
Sorry, something went wrong.
I assume he means that the gist of the current warning is better than the proposed alternative. As in, just fix the grammar error without changing the intent of what we're trying to say. |
Sorry, something went wrong.
Yes, the minor typographic error aside, the current is better information for the user. I think it would be better to correct the current message. |
Sorry, something went wrong.
What would be your suggestion? |
Sorry, something went wrong.
Sorry, something went wrong.
Sorry, something went wrong.
PR-URL: nodejs#49069 Fixes: nodejs#49026 Reviewed-By: Chemi Atlow <chemi@atlow.co.il> Reviewed-By: Jacob Smith <jacob@frende.me>
PR-URL: nodejs#49069 Fixes: nodejs#49026 Reviewed-By: Chemi Atlow <chemi@atlow.co.il> Reviewed-By: Jacob Smith <jacob@frende.me>
PR-URL: nodejs#49069 Fixes: nodejs#49026 Reviewed-By: Chemi Atlow <chemi@atlow.co.il> Reviewed-By: Jacob Smith <jacob@frende.me>
PR-URL: nodejs/node#49069 Backport-PR-URL: nodejs/node#50669 Fixes: nodejs/node#49026 Reviewed-By: Chemi Atlow <chemi@atlow.co.il> Reviewed-By: Jacob Smith <jacob@frende.me>
PR-URL: nodejs/node#49069 Backport-PR-URL: nodejs/node#50669 Fixes: nodejs/node#49026 Reviewed-By: Chemi Atlow <chemi@atlow.co.il> Reviewed-By: Jacob Smith <jacob@frende.me>
| Back | FazBrowse Home | New Git URL |
Fixes: #49026
Before:
After: