Skip to content

doc: add documentation for starting ci job via label - #35551

Closed
juanarbol wants to merge 1 commit into
nodejs:masterfrom
juanarbol:juanarbol/add-kickoff-ci-doc
Closed

doc: add documentation for starting ci job via label#35551
juanarbol wants to merge 1 commit into
nodejs:masterfrom
juanarbol:juanarbol/add-kickoff-ci-doc

Conversation

@juanarbol

Copy link
Copy Markdown
Member

Special thanks to @mmarchini, this is a really awesome feature!!

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

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/tsc

@nodejs-github-botnodejs-github-bot added the doc Issues and PRs related to the documentations. label Oct 7, 2020
Comment threaddoc/guides/collaborator-guide.md Outdated
Comment threaddoc/guides/collaborator-guide.md Outdated
Comment threaddoc/guides/collaborator-guide.md Outdated

@FlarnaFlarna 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.

Maybe add that request-ci label is removed automatically once the CI job has been started.

@mhdawsonmhdawson 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.

LGTM

@juanarbol
juanarbolforce-pushed the juanarbol/add-kickoff-ci-doc branch from 1a24062 to 5cfe307CompareOctober 13, 2020 23:22
juanarbol added a commit to juanarbol/node that referenced this pull request Oct 13, 2020
PR-URL: nodejs#35551
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Shelley Vohr <codebytere@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
@juanarbol

Copy link
Copy Markdown
MemberAuthor

Landed in d5e6495

MylesBorins pushed a commit that referenced this pull request Oct 14, 2020
PR-URL: #35551
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Shelley Vohr <codebytere@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
@MylesBorinsMylesBorins mentioned this pull request Oct 14, 2020
joesepi pushed a commit to joesepi/node that referenced this pull request Jan 8, 2021
PR-URL: nodejs#35551
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Shelley Vohr <codebytere@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
@juanarbol
juanarbol deleted the juanarbol/add-kickoff-ci-doc branch January 19, 2021 16:25
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants

@juanarbol@nodejs-github-bot@Trott@antsmartian@codebytere@gengjiawen@mmarchini@mhdawson@Flarna