|
| 1 | +# Contributing guidelines |
| 2 | + |
| 3 | +If you're willing to create a new PR on Starport Network, make sure that you read and comply with this document. |
| 4 | + |
| 5 | +Start a new [Discussion](https://github.com/tendermint/spn/discussions/new) if you want to propose changes to this document. |
| 6 | + |
| 7 | +Thank you for your contribution! |
| 8 | + |
| 9 | +## Providing Feedback |
| 10 | + |
| 11 | +* Before you open an issue, do a web search, and check for [existing open and closed GitHub Issues](https://github.com/tendermint/spn/issues) to see if your question has already been asked and answered. If you find a relevant topic, you can comment on that issue. |
| 12 | +* To provide feedback or ask a question, create a [GitHub issue](https://github.com/tendermint/spn/issues/new/choose). Be sure to provide the relevant information, case study, or informative links as suggested by the Pull Request template. |
| 13 | +* We recommend using GitHub issues for issues and feedback. However, you can ask quick questions on the [#🔨cosmos-sdk-starport](https://discord.com/channels/669268347736686612/737461683588431924) channel in Discord. |
| 14 | + |
| 15 | +## Opening pull requests |
| 16 | + |
| 17 | +### Choose a good PR title |
| 18 | + |
| 19 | +Avoid long names in your PR titles. Make sure your title has fewer than 60 characters. |
| 20 | + |
| 21 | +Follow [Conventional Commits](https://www.conventionalcommits.org/en/v1.0.0/) guidelines and keywords to find the best title. |
| 22 | + |
| 23 | +Use parentheses to identify the package or feature that you worked on. For example: `feat(services/chain)`, `fix(scaffolding)`. |
| 24 | + |
| 25 | +### Review your own code |
| 26 | + |
| 27 | +Make sure that you manually tested the changes you're introducing before creating a PR or pushing another commit. |
| 28 | + |
| 29 | +Monitor your PR to make sure that all CI checks pass and the PR shows **All checks have passed** (the checkmark is green). |
| 30 | + |
| 31 | +### Avoid rebasing commits in your branch |
| 32 | + |
| 33 | +Avoid rebasing after you open your PRs to reviews. Instead, add more commits to your PR. It's OK to do force pushes if the PR is still in draft mode and was never opened to reviews before. |
| 34 | + |
| 35 | +A reviewer likes to see a linear commit history while reviewing. If you tend to force push from an older commit, reviewer might lose track in your recent changes and will have to start reviewing from scratch. |
| 36 | + |
| 37 | +Don't worry about adding too many commits. The commits are squashed into a single commit while merging. Your PR title is used as the commit message. |
| 38 | + |
| 39 | +### Ask for help |
| 40 | + |
| 41 | +If you started a PR but couldn't finish it for whatever reason, don't give up. Instead, just ask for help. Someone else can take over and assume the ownership. |
| 42 | + |
| 43 | +We appreciate every bit of your work! |
| 44 | + |
| 45 | +## Coding style |
0 commit comments