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

build: extract common parts from addon .buildstamp by danbev · Pull Request #22171 · nodejs/node · GitHub

/ node Public

build: extract common parts from addon .buildstamp - #22171

Closed
danbev wants to merge 1 commit into
nodejs:masterfrom
danbev:build-addons-refactor
Closed

build: extract common parts from addon .buildstamp#22171
danbev wants to merge 1 commit into
nodejs:masterfrom
danbev:build-addons-refactor

Conversation

danbev commented Aug 7, 2018

Copy link
Copy Markdown
Contributor

This commit extracts common parts from the targets
test/addons/.buildstamp, and test/addons-napi/.buildstamp to reduce some
duplication.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

This commit extracts common parts from the targets
test/addons/.buildstamp, and test/addons-napi/.buildstamp to reduce some
duplication.

Copy link
Copy Markdown
Collaborator

nodejs-github-bot added the build Issues and PRs related to build files or the CI. label Aug 7, 2018

danbev commented Aug 7, 2018

Copy link
Copy Markdown
Contributor Author

Trott commented Aug 9, 2018

Copy link
Copy Markdown
Member

@nodejs/build-files

maclover7 added the author ready PRs that have at least one approval, no outstanding review comments, and a CI started. label Aug 9, 2018

danbev commented Aug 10, 2018

Copy link
Copy Markdown
Contributor Author

Landed in d3d54aa.

danbev closed this Aug 10, 2018
danbev deleted the build-addons-refactor branch August 10, 2018 03:46
danbev added a commit that referenced this pull request Aug 10, 2018
This commit extracts common parts from the targets
test/addons/.buildstamp, and test/addons-napi/.buildstamp to reduce some
duplication.

PR-URL: #22171
Reviewed-By: Jon Moss <me@jonathanmoss.me>
targos pushed a commit that referenced this pull request Aug 11, 2018
This commit extracts common parts from the targets
test/addons/.buildstamp, and test/addons-napi/.buildstamp to reduce some
duplication.

PR-URL: #22171
Reviewed-By: Jon Moss <me@jonathanmoss.me>
firass111 pushed a commit to firass111/Project_node1 that referenced this pull request Apr 16, 2025
This commit extracts common parts from the targets
test/addons/.buildstamp, and test/addons-napi/.buildstamp to reduce some
duplication.

PR-URL: nodejs/node#22171
Reviewed-By: Jon Moss <me@jonathanmoss.me>
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

author ready PRs that have at least one approval, no outstanding review comments, and a CI started. build Issues and PRs related to build files or the CI.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL