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.

Sign up for GitHub

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

Jump to bottom

doc: clarify when not to run CI on docs #33101

Closed
wants to merge 2 commits into from
Closed

doc: clarify when not to run CI on docs #33101

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
1b7d316
doc: clarify when not to run CI on docs
juanarbol Apr 27, 2020
cfea063
fixup
juanarbol Apr 29, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
5 changes: 3 additions & 2 deletions doc/guides/collaborator-guide.md
View file
Open in desktop
Expand Up @@ -177,8 +177,9 @@ All pull requests must pass continuous integration tests. Code changes must pass
on [project CI server](https://ci.nodejs.org/). Pull requests that only change
documentation and comments can use GitHub Actions results.

Do not land any pull requests without passing (green or yellow) CI runs. If
there are CI failures unrelated to the change in the pull request, try "Resume
Do not land any pull requests without passing (green or yellow) CI runs (except
for documentation-only changes, with green GitHub actions is enough). If there
are CI failures unrelated to the change in the pull request, try "Resume
addaleax marked this conversation as resolved.
Show resolved Hide resolved
Build". It is in the left navigation of the relevant `node-test-pull-request`
job. It will preserve all the green results from the current job but re-run
everything else. Start a fresh CI if more than seven days have elapsed since
Expand Down