Skip to content
This repository was archived by the owner on Feb 20, 2026. It is now read-only.

Revert "build!: upgrade engines field to >=8.10.0"#477

Merged
kalyanac merged 1 commit intomasterfrom
revert-476-update-engine-field-2-the-updates-revenge
May 2, 2019
Merged

Revert "build!: upgrade engines field to >=8.10.0"#477
kalyanac merged 1 commit intomasterfrom
revert-476-update-engine-field-2-the-updates-revenge

Conversation

@nolanmar511
Copy link
Copy Markdown
Contributor

@nolanmar511 nolanmar511 commented May 2, 2019

Reverts #476

#476 should not have been merged in.

It allows the profiler to run with versions of Node 10 impacted by https://bugs.chromium.org/p/chromium/issues/detail?id=847863 (gc can take several minutes when heap profiling is enabled)

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label May 2, 2019
@nolanmar511 nolanmar511 requested review from bcoe and kalyanac May 2, 2019 19:02
Copy link
Copy Markdown
Contributor

@JustinBeckwith JustinBeckwith left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apologies!

@kalyanac kalyanac merged commit 2453786 into master May 2, 2019
@nolanmar511 nolanmar511 deleted the revert-476-update-engine-field-2-the-updates-revenge branch May 2, 2019 19:45
@nolanmar511
Copy link
Copy Markdown
Contributor Author

No problems!

FYI to bcoe@ -- PR#476 has been reverted.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants