Skip to content

test: remove common.enoughTestCpu - #39161

Merged
Trott merged 1 commit into
nodejs:masterfrom
Trott:enough
Jul 4, 2021
Merged

test: remove common.enoughTestCpu#39161
Trott merged 1 commit into
nodejs:masterfrom
Trott:enough

Conversation

@Trott

Copy link
Copy Markdown
Member

All hosts in CI return true for common.enoughTestCpu. At least for our
CI, it is always true, so we can remove it.

@TrottTrott added the request-ci Add this label to start a Jenkins CI on a PR. label Jun 26, 2021
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jun 26, 2021
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@github-actionsgithub-actionsBot added needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels Jun 26, 2021
All hosts in CI return true for common.enoughTestCpu. At least for our
CI, it is always true, so we can remove it.
PR-URL: nodejs#39161
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@Trott

Trott commented Jul 4, 2021

Copy link
Copy Markdown
MemberAuthor

Landed in 32b6f4b

@Trott
Trott merged commit 32b6f4b into nodejs:masterJul 4, 2021
@Trott
Trott deleted the enough branch July 4, 2021 14:27
targos pushed a commit that referenced this pull request Jul 11, 2021
All hosts in CI return true for common.enoughTestCpu. At least for our
CI, it is always true, so we can remove it.
PR-URL: #39161
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@targostargos mentioned this pull request Jul 13, 2021
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-ciPRs that need a full CI run.testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@Trott@nodejs-github-bot@lpinca@targos