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

build: fix building embedtest in GN build by zcbenz · Pull Request #53145 · nodejs/node · GitHub

/ node Public

build: fix building embedtest in GN build - #53145

Closed
zcbenz wants to merge 1 commit into
nodejs:mainfrom
zcbenz:gn-embedtest
Closed

build: fix building embedtest in GN build#53145
zcbenz wants to merge 1 commit into
nodejs:mainfrom
zcbenz:gn-embedtest

Conversation

zcbenz commented May 25, 2024

Copy link
Copy Markdown
Contributor

Update the GN build config of embedtest to match the change in #52646.

zcbenz added the request-ci Add this label to start a Jenkins CI on a PR. label May 25, 2024
github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label May 25, 2024

Copy link
Copy Markdown
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/59398/

Copy link
Copy Markdown
Collaborator

Copy link
Copy Markdown
Collaborator

Copy link
Copy Markdown
Collaborator

zcbenz added the request-ci Add this label to start a Jenkins CI on a PR. label May 28, 2024
github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label May 28, 2024

Copy link
Copy Markdown
Collaborator

Copy link
Copy Markdown
Collaborator

Copy link
Copy Markdown
Collaborator

Copy link
Copy Markdown
Collaborator

Copy link
Copy Markdown
Collaborator

zcbenz added a commit that referenced this pull request May 28, 2024
PR-URL: #53145
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>

zcbenz commented May 28, 2024

Copy link
Copy Markdown
Contributor Author

Landed in 067ab06

zcbenz closed this May 28, 2024
zcbenz deleted the gn-embedtest branch May 28, 2024 10:24
targos pushed a commit that referenced this pull request Jun 1, 2024
PR-URL: #53145
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
RafaelGSS mentioned this pull request Jun 7, 2024
soophoo pushed a commit to soophoo/node that referenced this pull request Jun 20, 2024
PR-URL: nodejs#53145
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
marco-ippolito pushed a commit that referenced this pull request Jul 19, 2024
PR-URL: #53145
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Luigi Pinca <luigipinca@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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL