Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: log statement is for devDir not nodedir #2840

Merged
merged 1 commit into from
Jun 6, 2023

Conversation

dsanders11
Copy link
Contributor

Checklist
  • npm install && npm test passes
  • tests are included
  • documentation is changed or added
  • commit message follows commit guidelines
Description of change

Updating this log statement to be accurate.

Copy link
Member

@rvagg rvagg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah, you're right I think, good catch

Signed-off-by: David Sanders <dsanders11@ucsbalum.com>
@dsanders11
Copy link
Contributor Author

@cclauss, this should be ready for merging, rebased on main and everything is green.

@cclauss cclauss merged commit 55048f8 into nodejs:main Jun 6, 2023
@dsanders11 dsanders11 deleted the patch-1 branch June 6, 2023 05:43
cclauss pushed a commit to cclauss/node-gyp that referenced this pull request Jun 13, 2023
Signed-off-by: David Sanders <dsanders11@ucsbalum.com>
lukekarrys pushed a commit that referenced this pull request Dec 2, 2024
Signed-off-by: David Sanders <dsanders11@ucsbalum.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants