Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

doc: fix the PR template CONTRIBUTING.md link #349

Merged
merged 1 commit into from
Feb 20, 2017

Conversation

gibfahn
Copy link
Member

@gibfahn gibfahn commented Feb 16, 2017

Using ./CONTRIBUTING.md in the PR template resolves to
https://github.com/nodejs/citgm/compare/CONTRIBUTING.md. Use the full path
instead (as is done in Node.js core).

Note that the CONTRIBUTING.md file hasn't yet been added (the PR to add it is
#314).

Link to core PR template: https://raw.githubusercontent.com/nodejs/node/master/.github/PULL_REQUEST_TEMPLATE.md

Checklist
  • documentation is changed or added
  • contribution guidelines followed here
  • commit message follows commit guidelines

gdams

This comment was marked as off-topic.

@gibfahn
Copy link
Member Author

gibfahn commented Feb 16, 2017

I'd say it doesn't need to wait for #314 , there's already a link to CONTRIBUTING.md in the PULL_REQUEST template, this just fixes the link.

gdams

This comment was marked as off-topic.

gdams

This comment was marked as off-topic.

mhdawson

This comment was marked as off-topic.

@gdams
Copy link
Member

gdams commented Feb 17, 2017

@gibfahn think this is good to land

Using ./CONTRIBUTING.md in the PR template resolves to
https://github.com/nodejs/citgm/compare/CONTRIBUTING.md. Use the full path
instead (as is done in Node.js core).

Note that the CONTRIBUTING.md file hasn't yet been added (the PR to add it is
nodejs#314).
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.

3 participants