Skip to content

doc: fixing the path to postMessage - #25332

Closed
mitar wants to merge 1 commit into
nodejs:masterfrom
mitar:patch-1
Closed

doc: fixing the path to postMessage#25332
mitar wants to merge 1 commit into
nodejs:masterfrom
mitar:patch-1

Conversation

@mitar

@mitarmitar commented Jan 3, 2019

Copy link
Copy Markdown
Contributor
Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines

@nodejs-github-botnodejs-github-bot added the doc Issues and PRs related to the documentations. label Jan 3, 2019

@addaleaxaddaleax left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Can you wrap this line at 80 characters?

@addaleaxaddaleax added the worker Issues and PRs related to Worker support. label Jan 7, 2019
@mitar

mitar commented Jan 8, 2019

Copy link
Copy Markdown
ContributorAuthor

Done.

@addaleax

Copy link
Copy Markdown
Member

@addaleaxaddaleax added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jan 8, 2019
@addaleax

Copy link
Copy Markdown
Member

Oh, I think the linter in Travis is failing because the link definition should also be changed?

@addaleaxaddaleax removed the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jan 8, 2019
@mitar

mitar commented Jan 8, 2019

Copy link
Copy Markdown
ContributorAuthor

You are right. This is anchor link or something.

@mitar

mitar commented Jan 8, 2019

Copy link
Copy Markdown
ContributorAuthor

I think I fixed it now.

@vsemozhetbyt

Copy link
Copy Markdown
Contributor

@vsemozhetbyt

Copy link
Copy Markdown
Contributor

Landed in 56e7a7c
Thank you!

pullBot pushed a commit to Rachelmorrell/node that referenced this pull request Jan 11, 2019
PR-URL: nodejs#25332
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
addaleax pushed a commit that referenced this pull request Jan 14, 2019
PR-URL: #25332
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
@BridgeARBridgeAR mentioned this pull request Jan 16, 2019
BridgeAR pushed a commit to BridgeAR/node that referenced this pull request Jan 16, 2019
PR-URL: nodejs#25332
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
@MylesBorinsMylesBorins mentioned this pull request Jan 24, 2019
BethGriggs pushed a commit that referenced this pull request Apr 28, 2019
PR-URL: #25332
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
@BethGriggsBethGriggs mentioned this pull request May 1, 2019
BethGriggs pushed a commit that referenced this pull request May 10, 2019
PR-URL: #25332
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
MylesBorins pushed a commit that referenced this pull request May 16, 2019
PR-URL: #25332
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docIssues and PRs related to the documentations.workerIssues and PRs related to Worker support.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@mitar@addaleax@vsemozhetbyt@jasnell@nodejs-github-bot