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

tools,doc: fix broken link in module.html #35446

Closed
wants to merge 0 commits into from

Conversation

Trott
Copy link
Member

@Trott Trott commented Oct 1, 2020

Update tools/doc/type-parser.js with correct URL for the
module.SourceMap class. This fixes a broken link in our documentation.

Checklist

@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to the documentations. tools Issues and PRs related to the tools directory. labels Oct 1, 2020
@jasnell
Copy link
Member

jasnell commented Oct 1, 2020

There shouldn't be any reason this has to wait to land. fast-track?

@jasnell jasnell added the fast-track PRs that do not need to wait for 48 hours to land. label Oct 1, 2020
@richardlau richardlau added the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 1, 2020
@github-actions github-actions bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 1, 2020
@github-actions
Copy link
Contributor

github-actions bot commented Oct 1, 2020

Commit Queue failed
- Loading data for nodejs/node/pull/35446
✔  Done loading data for nodejs/node/pull/35446
----------------------------------- PR info ------------------------------------
Title      tools,doc: fix broken link in module.html (#35446)
Author     Rich Trott  (@Trott)
Branch     Trott:fix-broken-sourcemap-link -> nodejs:master
Labels     doc, fast-track, tools
Commits    1
 - tools,doc: fix broken link in module.html
Committers 1
 - Rich Trott 
PR-URL: https://github.com/nodejs/node/pull/35446
Reviewed-By: James M Snell 
Reviewed-By: Richard Lau 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/35446
Reviewed-By: James M Snell 
Reviewed-By: Richard Lau 
--------------------------------------------------------------------------------
   ✔  Last GitHub Actions successful
   ✖  No Jenkins CI runs detected
   ℹ  This PR was created on Thu, 01 Oct 2020 13:59:27 GMT
   ✔  Approvals: 2
   ✔  - James M Snell (@jasnell) (TSC): https://github.com/nodejs/node/pull/35446#pullrequestreview-500443557
   ✔  - Richard Lau (@richardlau): https://github.com/nodejs/node/pull/35446#pullrequestreview-500539970
   ℹ  This PR is being fast-tracked
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu

@github-actions github-actions bot added the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Oct 1, 2020
@richardlau richardlau added request-ci Add this label to start a Jenkins CI on a PR. and removed commit-queue-failed An error occurred while landing this pull request using GitHub Actions. labels Oct 1, 2020
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Oct 1, 2020
@nodejs-github-bot
Copy link
Collaborator

@richardlau richardlau added the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 2, 2020
@github-actions github-actions bot added commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Oct 2, 2020
Trott added a commit to Trott/io.js that referenced this pull request Oct 3, 2020
Update tools/doc/type-parser.js with correct URL for the
module.SourceMap class. This fixes a broken link in our documentation.

PR-URL: nodejs#35446
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
@Trott Trott closed this Oct 3, 2020
@Trott Trott deleted the fix-broken-sourcemap-link branch October 3, 2020 12:43
@Trott
Copy link
Member Author

Trott commented Oct 3, 2020

Landed in e9639ee

danielleadams pushed a commit that referenced this pull request Oct 6, 2020
Update tools/doc/type-parser.js with correct URL for the
module.SourceMap class. This fixes a broken link in our documentation.

PR-URL: #35446
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
@danielleadams danielleadams mentioned this pull request Oct 6, 2020
joesepi pushed a commit to joesepi/node that referenced this pull request Jan 8, 2021
Update tools/doc/type-parser.js with correct URL for the
module.SourceMap class. This fixes a broken link in our documentation.

PR-URL: nodejs#35446
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
@targos targos removed the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Sep 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Issues and PRs related to the documentations. fast-track PRs that do not need to wait for 48 hours to land. tools Issues and PRs related to the tools directory.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants