Skip to content

doc: add boilerplate files to repo - #213

Merged
mhdawson merged 3 commits into
nodejs:mainfrom
mhdawson:add-boilerplate-files
Aug 4, 2023
Merged

mhdawson merged 3 commits into
nodejs:mainfrom
mhdawson:add-boilerplate-files

Conversation

@mhdawson

Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: Michael Dawson <mdawson@devrus.com>

@KevinEady KevinEady left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

See comments regarding license.

Secondly, should we also add the PR issue template https://raw.githubusercontent.com/nodejs/node-addon-api/main/.github/PULL_REQUEST_TEMPLATE.md ? It's the same contents as CONTRIBUTING.md, but will show the text in any new PR created.

Comment thread LICENSE.md Outdated
mhdawson and others added 2 commits July 14, 2023 10:47
Co-authored-by: Kevin Eady <8634912+KevinEady@users.noreply.github.com>
Signed-off-by: Michael Dawson <mdawson@devrus.com>
@mhdawson

Copy link
Copy Markdown
Member Author

@KevinEady good catch added .PULL_REQUEST_TEMPLATE.md and accepted suggestion to licence.

@mhdawson

mhdawson commented Aug 3, 2023

Copy link
Copy Markdown
Member Author

@KevinEady if you are checking GitHub could you have a re-review so I can land this one?

@KevinEady KevinEady left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@mhdawson
mhdawson merged commit 4949af1 into nodejs:main Aug 4, 2023
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.

2 participants