Skip to content

Conversation

@Trott
Copy link
Member

@Trott Trott commented Jun 4, 2020

Disable linting for prohibited strings in the collaborator list so that
sam-github does not have to be sam-GitHub.

Refs: nodejs/remark-preset-lint-node#96

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • documentation is changed or added
  • commit message follows commit guidelines

@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Jun 4, 2020
@Trott Trott force-pushed the disable-for-collab-list branch 2 times, most recently from 60a8ba0 to 03bea59 Compare June 4, 2020 05:04
@Trott
Copy link
Member Author

Trott commented Jun 4, 2020

Would like to fast track this because nodejs/remark-preset-lint-node#96 is blocked until this lands. Collaborators, please 👍 here to fast-track.

@Trott Trott added the fast-track PRs that do not need to wait for 48 hours to land. label Jun 4, 2020
@Trott Trott requested review from cjihrig and richardlau June 4, 2020 05:05
`common.gypi` is used by `node-gyp` to compile addons. Default values
must be provided for variables that may not exist on older versions of
Node.js so that older versions of Node.js can be used to compile addons
for later versions of Node.js.

Add default values for `v8_enable_pointer_compression` and
`v8_enable_31bit_smis_on_64bit_arch`.

PR-URL: nodejs#33688
Refs: nodejs#30463
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Signed-off-by: Richard Lau <riclau@uk.ibm.com>
Disable linting for prohibited strings in the collaborator list so that
sam-github does not have to be sam-GitHub.

Refs: nodejs/remark-preset-lint-node#96
Signed-off-by: Rich Trott <rtrott@gmail.com>

PR-URL: nodejs#33719
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Beth Griggs <Bethany.Griggs@uk.ibm.com>
@Trott Trott merged commit 25d48a6 into nodejs:master Jun 4, 2020
@Trott Trott deleted the disable-for-collab-list branch June 4, 2020 12:16
codebytere pushed a commit that referenced this pull request Jun 18, 2020
Disable linting for prohibited strings in the collaborator list so that
sam-github does not have to be sam-GitHub.

Refs: nodejs/remark-preset-lint-node#96
Signed-off-by: Rich Trott <rtrott@gmail.com>

PR-URL: #33719
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Beth Griggs <Bethany.Griggs@uk.ibm.com>
@codebytere codebytere mentioned this pull request Jun 28, 2020
codebytere pushed a commit that referenced this pull request Jun 30, 2020
Disable linting for prohibited strings in the collaborator list so that
sam-github does not have to be sam-GitHub.

Refs: nodejs/remark-preset-lint-node#96
Signed-off-by: Rich Trott <rtrott@gmail.com>

PR-URL: #33719
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Beth Griggs <Bethany.Griggs@uk.ibm.com>
codebytere pushed a commit that referenced this pull request Jul 9, 2020
Disable linting for prohibited strings in the collaborator list so that
sam-github does not have to be sam-GitHub.

Refs: nodejs/remark-preset-lint-node#96
Signed-off-by: Rich Trott <rtrott@gmail.com>

PR-URL: #33719
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Beth Griggs <Bethany.Griggs@uk.ibm.com>
@codebytere codebytere mentioned this pull request Jul 13, 2020
codebytere pushed a commit that referenced this pull request Jul 14, 2020
Disable linting for prohibited strings in the collaborator list so that
sam-github does not have to be sam-GitHub.

Refs: nodejs/remark-preset-lint-node#96
Signed-off-by: Rich Trott <rtrott@gmail.com>

PR-URL: #33719
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Beth Griggs <Bethany.Griggs@uk.ibm.com>
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants