Skip to content

deps: fix include_dirs of nbytes - #53862

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
zcbenz:nbytes-include-dir
Jul 18, 2024
Merged

deps: fix include_dirs of nbytes#53862
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
zcbenz:nbytes-include-dir

Conversation

@zcbenz

Copy link
Copy Markdown
Contributor

Set correct include dirs for both GYP and GN builds.

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/gyp
  • @nodejs/security-wg

@nodejs-github-botnodejs-github-bot added dependencies Pull requests that update a dependency file. needs-ci PRs that need a full CI run. labels Jul 16, 2024
@zcbenzzcbenz added the request-ci Add this label to start a Jenkins CI on a PR. label Jul 18, 2024
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jul 18, 2024
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@zcbenzzcbenz added the commit-queue Add this label to land a pull request using GitHub Actions. label Jul 18, 2024
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jul 18, 2024
@nodejs-github-bot
nodejs-github-bot merged commit 014dad5 into nodejs:mainJul 18, 2024
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 014dad5

ehsankhfr pushed a commit to ehsankhfr/node that referenced this pull request Jul 18, 2024
PR-URL: nodejs#53862
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
targos pushed a commit that referenced this pull request Jul 28, 2024
PR-URL: #53862
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@RafaelGSSRafaelGSS mentioned this pull request Jul 30, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependenciesPull requests that update a dependency file.needs-ciPRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@zcbenz@nodejs-github-bot@jasnell@lpinca@anonrig@gengjiawen@targos