| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
|
Node.js Collaborators, please, add 👍 here if you approve fast-tracking. |
Sorry, something went wrong.
Sorry, something went wrong.
There was a problem hiding this comment.
Confirmed that links appear correctly in net.md in private branch
Sorry, something went wrong.
|
@vsemozhetbyt just as a question: did you check intermediate versions as well? |
Sorry, something went wrong.
|
@BridgeAR |
Sorry, something went wrong.
|
Landed in 536b1fb |
Sorry, something went wrong.
PR-URL: #20411 Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
PR-URL: #20411 Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
| Back | FazBrowse Home | New Git URL |
Checklist
Status quo
3cb8e64 + 647954d have fixed a method signature, but caused link rendering issues in 2 places of the HTML doc:
see this nightly doc section or a screenshot with the issue in red frame.
see this nightly doc section or a screenshot with the issue in red frame.
In the GitHub docs, both links are rendered properly: see here and here.
The cause
We have rather an old version of marked module and it seems it has a parsing bug for this pattern:
Minimal reproduction:
In this case, marked renders the first link part as a code fragment in non-code square brackets, and then renders the second part as a link inferring both the text and the URL from it.
What can we do
The bug is fixed in marked master. Compare the output from the tip-of-tree:
But it seems we cannot update now as marked is still unstable, has possible new bugs and experiences turbulent renewal (see issue with a comment).
So we can use this workaround till marked 1.0 is released. See screenshots of fixed links and here and here.
If we should add any HTML comments in these places with PR URL or TODO instructions, please, suggest the wording and format.
cc @nodejs/documentation