Skip to content

test: update test to comply with lint rule - #19784

Closed
Trott wants to merge 1 commit into
nodejs:masterfrom
Trott:unbreak-ci
Closed

test: update test to comply with lint rule#19784
Trott wants to merge 1 commit into
nodejs:masterfrom
Trott:unbreak-ci

Conversation

@Trott

@Trott Trott commented Apr 4, 2018

Copy link
Copy Markdown
Member

Ref: #19502 (comment)

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

@Trott Trott added the fast-track PRs proposed for a shorter-than-standard waiting period before landing. label Apr 4, 2018
@nodejs-github-bot nodejs-github-bot added the test Issues and PRs related to Node.js core tests and test infrastructure. label Apr 4, 2018
@Trott

Trott commented Apr 4, 2018

Copy link
Copy Markdown
Member Author

Lite CI: https://ci.nodejs.org/job/node-test-pull-request-lite/413/

(Lite CI because it only touches a comment and no executable code.)

@Trott

Trott commented Apr 4, 2018

Copy link
Copy Markdown
Member Author

@nodejs/collaborators Some fast-track approvals for this to unbreak CI would be great!

@Trott

Trott commented Apr 4, 2018

Copy link
Copy Markdown
Member Author

(CI is green.)

@Trott
Trott requested a review from a team April 4, 2018 02:24
@Trott

Trott commented Apr 4, 2018

Copy link
Copy Markdown
Member Author

Landed in 8891072

@Trott Trott closed this Apr 4, 2018
Trott added a commit to Trott/io.js that referenced this pull request Apr 4, 2018
Ref: nodejs#19502 (comment)

PR-URL: nodejs#19784
Reviewed-By: Teddy Katz <teddy.katz@gmail.com>
Reviewed-By: Matheus Marchini <matheus@sthima.com>
targos pushed a commit that referenced this pull request Apr 4, 2018
Ref: #19502 (comment)

PR-URL: #19784
Reviewed-By: Teddy Katz <teddy.katz@gmail.com>
Reviewed-By: Matheus Marchini <matheus@sthima.com>
@targos targos mentioned this pull request Apr 4, 2018
@Trott
Trott deleted the unbreak-ci branch January 13, 2022 22:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fast-track PRs proposed for a shorter-than-standard waiting period before landing. test Issues and PRs related to Node.js core tests and test infrastructure.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants