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.

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

fix: generate valid config.gypi #31443

Merged
merged 2 commits into from Oct 18, 2021
Merged

fix: generate valid config.gypi #31443

merged 2 commits into from Oct 18, 2021

Conversation

zcbenz
Copy link
Member

@zcbenz zcbenz commented Oct 15, 2021

Backport of #31404.

Notes: Generate valid config.gypi file in Node.js headers.

@zcbenz zcbenz requested a review from as a code owner Oct 15, 2021
@electron-cation electron-cation bot added the new-pr 🌱 label Oct 15, 2021
@trop trop bot mentioned this pull request Oct 15, 2021
@trop trop bot added 14-x-y backport semver/patch labels Oct 15, 2021
@electron-cation electron-cation bot removed the new-pr 🌱 label Oct 15, 2021
ckerr
ckerr approved these changes Oct 18, 2021
@ckerr
Copy link
Member

@ckerr ckerr commented Oct 18, 2021

electron-woa-testing failure looks unrelated

@ckerr ckerr merged commit c5a7ccc into 14-x-y Oct 18, 2021
14 of 15 checks passed
@ckerr ckerr deleted the valid-config-gypi-14 branch Oct 18, 2021
@release-clerk
Copy link

@release-clerk release-clerk bot commented Oct 18, 2021

Release Notes Persisted

Generate valid config.gypi file in Node.js headers.

@jkleinsc
Copy link
Contributor

@jkleinsc jkleinsc commented Nov 24, 2021

/trop run backport-to 13-x-y

@trop
Copy link
Contributor

@trop trop bot commented Nov 24, 2021

The backport process for this PR has been manually initiated - sending your PR to 13-x-y!

@trop
Copy link
Contributor

@trop trop bot commented Nov 24, 2021

I have automatically backported this PR to "13-x-y", please check out #31989

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
14-x-y backport semver/patch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants