Skip to content

Commit

Permalink
Merge pull request #356 from jamestalmage/seek-early-feedback
Browse files Browse the repository at this point in the history
contributing.md - seek early feedback
  • Loading branch information
sindresorhus committed Dec 23, 2015
2 parents ae446f0 + 94c0f9b commit 65ae07c
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions contributing.md
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ We have a [chat](https://gitter.im/sindresorhus/ava). Jump in there and lurk, ta
### Submitting a pull request

- Non-trivial changes are often best discussed in an issue first, to prevent you from doing unnecessary work.
- For ambitious tasks, you should try to get your work in front of the community for feedback as soon as possible. Open a pull request as soon as you have done the minimum needed to demonstrate your idea. At this early stage, don't worry about making things perfect, or 100% complete. Add a [WIP] prefix to the title, and describe what you still need to do. This lets reviewers know not to nit-pick small details or point out improvements you already know you need to make.
- New features should be accompanied with tests and documentation.
- Don't include unrelated changes.
- Lint and test before submitting the pull request by running `$ npm test`.
Expand Down

0 comments on commit 65ae07c

Please sign in to comment.