👍🎉 First off, thanks for taking the time to contribute! 🎉👍
The following is a set of guidelines for contributing to walletconnect-dart-sdk. These are mostly guidelines, not rules. Use your best judgment, and feel free to propose changes to this document in a pull request.
If you intend to change the public API, or make any non-trivial changes to the implementation, we recommend filing an issue. This lets us reach an agreement on your proposal before you put significant effort into it.
If you’re only fixing a bug, it’s fine to submit a pull request right away but we still recommend to file an issue detailing what you’re fixing. This is helpful in case we don’t accept that specific fix but want to keep track of the issue.
Before creating a pull request please:
- Fork the repository and create your branch from
master
. - Install all dependencies (
flutter packages get
orpub get
). - Squash your commits and ensure you have a meaningful commit message.
- If you’ve fixed a bug or added code that should be tested, add tests! Pull Requests without 100% test coverage will not be approved.
- Ensure the test suite passes.
- If you've changed the public API, make sure to update/add documentation.
- Format your code (
dartfmt -w .
). - Analyze your code (
dartanalyzer --fatal-infos --fatal-warnings .
). - Create the Pull Request.
- Verify that all status checks are passing.
While the prerequisites above must be satisfied prior to having your pull request reviewed, the reviewer(s) may ask you to complete additional design work, tests, or other changes before your pull request can be ultimately accepted.
Examples live in the examples
folder.
- For a Flutter example, add it to the
folder
list in theexamples-flutter
step. - For a web example, add it to the
folder
list in theexamples-web
step. - For a pure Dart example, add it to the
folder
list in theexamples-pure
step.
If you want to just ask a question or get feedback on an idea you can post it on Discord.
By contributing to walletconnect-dart-sdk, you agree that your contributions will be licensed under its MIT license.