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

gyp: put cctest filenames in variables by zcbenz · Pull Request #49178 · nodejs/node · GitHub

/ node Public

gyp: put cctest filenames in variables - #49178

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
zcbenz:patch-10
Sep 8, 2023
Merged

gyp: put cctest filenames in variables#49178
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
zcbenz:patch-10

Conversation

zcbenz commented Aug 15, 2023

Copy link
Copy Markdown
Contributor

This is a followup to #46965, which will be very helpful for #47637.

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/gyp

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 Aug 15, 2023
joyeecheung added the request-ci Add this label to start a Jenkins CI on a PR. label Aug 15, 2023
github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Aug 15, 2023

Copy link
Copy Markdown
Collaborator

joyeecheung added the request-ci Add this label to start a Jenkins CI on a PR. label Aug 21, 2023
github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Aug 21, 2023

Copy link
Copy Markdown
Collaborator

Copy link
Copy Markdown
Collaborator

Qard added the commit-queue Add this label to land a pull request using GitHub Actions. label Sep 8, 2023
nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Sep 8, 2023
nodejs-github-bot merged commit 3163f8d into nodejs:main Sep 8, 2023

Copy link
Copy Markdown
Collaborator

Landed in 3163f8d

ruyadorno pushed a commit that referenced this pull request Sep 28, 2023
PR-URL: #49178
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Stephen Belanger <admin@stephenbelanger.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

build Issues and PRs related to build files or the CI. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants


Back | FazBrowse Home | New Git URL