FoundationPress icon indicating copy to clipboard operation
FoundationPress copied to clipboard

Add PULL_REQUEST_TEMPLATE.md & CONTRIBUTING.md files

Open colin-marshall opened this issue 6 years ago • 2 comments

@Aetles made some great points regarding documentation of commits in https://github.com/olefredrik/FoundationPress/issues/1123#issuecomment-341810747. I think one thing that would help improve this is to add .github/PULL_REQUEST_TEMPLATE.md with some basic info requirements for every pull request. More info: https://help.github.com/articles/creating-a-pull-request-template-for-your-repository/

Along those same lines, we should also add .github/CONTRIBUTING.md to the repo. More info: https://help.github.com/articles/setting-guidelines-for-repository-contributors/

What do you guys feel we should put in these files?

colin-marshall avatar Nov 20 '17 20:11 colin-marshall

Great initiative! I think this would be useful indeed. As a rule of thumb, I think that all pull requests should be linked to an issue. That way, we want to make sure that the community gets an opportunity to make feedback before potentially unexpected changes suddenly end up in the master branch. This will apply to bug fixes as well as introduction of new features.

(I'm the first to admit that I need to get better on this. As there have been no written guidelines for this, I have not always followed my own best practices.)

Consise and descriptive commit messages should also be written down as part of the guidelines.

Do you agree? Do you have anything specific in mind?

olefredrik avatar Nov 21 '17 17:11 olefredrik

@olefredrik I think an issue corresponding to every pull request is a great idea. That and directions for more informative commit messages will be a great start.

colin-marshall avatar Nov 28 '17 17:11 colin-marshall