FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

test: increase memory for coverage action by bcoe · Pull Request #39690 · nodejs/node · GitHub

/ node Public

test: increase memory for coverage action - #39690

Closed
bcoe wants to merge 1 commit into
nodejs:masterfrom
bcoe:increase-memory-coverage
Closed

test: increase memory for coverage action#39690
bcoe wants to merge 1 commit into
nodejs:masterfrom
bcoe:increase-memory-coverage

Conversation

bcoe commented Aug 6, 2021

Copy link
Copy Markdown
Contributor

We already set --max-old-space-size=8192 for Windows.

Fixes #39445

bcoe requested a review from targos August 6, 2021 21:47
bcoe added the author ready PRs that have at least one approval, no outstanding review comments, and a CI started. label Aug 6, 2021
nodejs-github-bot added the meta Issues and PRs related to the general management of the project. label Aug 6, 2021
bcoe added fast-track PRs that do not need to wait for 48 hours to land. and removed meta Issues and PRs related to the general management of the project. labels Aug 6, 2021

github-actions Bot commented Aug 6, 2021

Copy link
Copy Markdown
Contributor

Fast-track has been requested by @bcoe. Please 👍 to approve.

richardlau added the commit-queue Add this label to land a pull request using GitHub Actions. label Aug 7, 2021
github-actions Bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Aug 7, 2021

github-actions Bot commented Aug 7, 2021

Copy link
Copy Markdown
Contributor

Landed in 822f9ff...5a01efd

github-actions Bot closed this Aug 7, 2021
nodejs-github-bot pushed a commit that referenced this pull request Aug 7, 2021
PR-URL: #39690
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
danielleadams pushed a commit that referenced this pull request Aug 16, 2021
PR-URL: #39690
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no outstanding review comments, and a CI started. fast-track PRs that do not need to wait for 48 hours to land.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

coverage-linux workflow going out of memory

6 participants


Back | FazBrowse Home | New Git URL