Skip to content

doc: replace node-test-pull-request-lite-pipeline from onboarding - #32736

Closed
juanarbol wants to merge 1 commit into
nodejs:masterfrom
juanarbol:juanarbol/onboarding-changes
Closed

doc: replace node-test-pull-request-lite-pipeline from onboarding#32736
juanarbol wants to merge 1 commit into
nodejs:masterfrom
juanarbol:juanarbol/onboarding-changes

Conversation

@juanarbol

Copy link
Copy Markdown
Member

Replace the obsolete node-test-pull-request-lite-pipeline CI
task in favour of node-test-pull-request-pipeline.

I've updated the docs based on this comment

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 Apr 9, 2020
@himself65
himself65 requested a review from TrottApril 9, 2020 05:50
Comment threadonboarding.md Outdated
Replace the obsolete node-test-pull-request-lite-pipeline CI
task in favour of node-test-pull-request.
@juanarbol
juanarbolforce-pushed the juanarbol/onboarding-changes branch from 8c7825e to 12b7f00CompareApril 9, 2020 13:39
Comment threadonboarding.md
@himself65himself65 added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Apr 11, 2020
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@Trott

Copy link
Copy Markdown
Member

CI: https://ci.nodejs.org/job/node-test-pull-request/30638/

Because this is a documentation-only change, it does not need to pass Jenkins. Passing the GitHub Actions tests is sufficient per https://github.com/nodejs/node/blob/master/doc/guides/collaborator-guide.md#testing-and-ci.

@Trott

Copy link
Copy Markdown
Member

Landed in 5c53538

@TrottTrott closed this Apr 11, 2020
Trott pushed a commit that referenced this pull request Apr 11, 2020
Replace the obsolete node-test-pull-request-lite-pipeline CI
task in favour of node-test-pull-request.
PR-URL: #32736
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Zeyu Yang <himself65@outlook.com>
targos pushed a commit that referenced this pull request Apr 13, 2020
Replace the obsolete node-test-pull-request-lite-pipeline CI
task in favour of node-test-pull-request.
PR-URL: #32736
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Zeyu Yang <himself65@outlook.com>
@targostargos mentioned this pull request Apr 13, 2020
BethGriggs pushed a commit that referenced this pull request Apr 14, 2020
Replace the obsolete node-test-pull-request-lite-pipeline CI
task in favour of node-test-pull-request.
PR-URL: #32736
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Zeyu Yang <himself65@outlook.com>
targos pushed a commit that referenced this pull request Apr 22, 2020
Replace the obsolete node-test-pull-request-lite-pipeline CI
task in favour of node-test-pull-request.
PR-URL: #32736
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Zeyu Yang <himself65@outlook.com>
@targostargos mentioned this pull request Apr 22, 2020
@juanarbol
juanarbol deleted the juanarbol/onboarding-changes branch January 19, 2021 16:26
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.docIssues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@juanarbol@nodejs-github-bot@Trott@jasnell@addaleax@richardlau@himself65