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

Revert "build: only generate specified build type files" by legendecas · Pull Request #53580 · nodejs/node · GitHub

/ node Public

Revert "build: only generate specified build type files" - #53580

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
legendecas:build/revert-ninja
Jun 26, 2024
Merged

Revert "build: only generate specified build type files"#53580
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
legendecas:build/revert-ninja

Conversation

Copy link
Copy Markdown
Member

This reverts commit 6cb940a.

Refs: #53511 (comment)

I think an alternative should be making the generated config.gypi works in both Release mode and Debug mode.

legendecas requested a review from targos June 25, 2024 09:04
nodejs-github-bot added build Issues and PRs related to build files or the CI. needs-ci PRs that need a full CI run. labels Jun 25, 2024
targos added the dont-land-on-v22.x PRs that should not land on the v22.x-staging branch and should not be released in v22.x. label Jun 25, 2024
legendecas added the request-ci Add this label to start a Jenkins CI on a PR. label Jun 25, 2024
github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jun 25, 2024

Copy link
Copy Markdown
Collaborator

legendecas added the fast-track PRs that do not need to wait for 48 hours to land. label Jun 25, 2024

Copy link
Copy Markdown
Contributor

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

Copy link
Copy Markdown
Member Author

Fast-tracking the revert to fix the ninja workflow composing with make.

Copy link
Copy Markdown
Collaborator

Copy link
Copy Markdown
Collaborator

legendecas added the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 26, 2024
nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 26, 2024
nodejs-github-bot merged commit ab5e58b into nodejs:main Jun 26, 2024

Copy link
Copy Markdown
Collaborator

Landed in ab5e58b

legendecas deleted the build/revert-ninja branch June 26, 2024 09:51
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

build Issues and PRs related to build files or the CI. dont-land-on-v22.x PRs that should not land on the v22.x-staging branch and should not be released in v22.x. fast-track PRs that do not need to wait for 48 hours to land. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants


Back | FazBrowse Home | New Git URL