👍🎉 First off, thanks for taking the time to contribute! 🎉👍
The following is a set of guidelines for contributing to fendersbot, which is hosted at fendersperth/fendersbot on GitHub. These are just guidelines, not rules, use your best judgment and feel free to propose changes to this document in a pull request.
Before creating enhancement suggestions, please check this list as you might find out that you don't need to create one. When you are creating an enhancement suggestion, please include as many details as possible.
- Perform a cursory search to see if the enhancement has already been suggested. If it has, add a comment to the existing issue instead of opening a new one.
Enhancement suggestions are tracked as GitHub issues. When you create an issue try and provide the following information where appropriate:
- Use a clear and descriptive title for the issue to identify the suggestion.
- Provide a step-by-step description of the suggested enhancement in as many details as possible.
- Provide specific examples to demonstrate the steps. Include copy/pasteable snippets which you use in those examples, as Markdown code blocks.
- Describe the current behavior and explain which behavior you expected to see instead and why.
- Explain why this enhancement would be useful to most fenders users.
- List some other bots or slack channels where this enhancement exists.
Unsure where to begin contributing to fendersbot? You can start by looking through these beginner
and help-wanted
issues:
- Beginner issues - issues which should only require a few lines of code, and a test or two.
- Help wanted issues - issues which should be a bit more involved than
beginner
issues.
Pull requests should come from topic branches on forks of the fendersbot repo and target the master branch.
Once the pull request receives a +1 from a core contributor it is ready to be merged to master.
If the pull request has come from a core contributor it is their responsibility to merge.
Try and use the following guidelines when creating pull requests.
- Follow the Javascript styleguide with the exception of no trailing commas.
- End files with a newline.
- Include thoughtfully-worded, well-structured Mocha bdd style tests in the
./test
folder. Run them usingnpm test
. - Avoid platform-dependent code.
We have jscs rules in the package.json based on the airbnb style guide.
The only exception from the airbnb guide is that we have turned off requireTrailingComma.
There are friendly jscs packages available for many editing tools.
- Include thoughtfully-worded, well-structured Mocha bdd style tests in the
./test
folder. - Run them using
npm test
.