Thank you for your interest in contributing to Solana Pay! All contributions are welcome no matter how big or small. This includes (but is not limited to) filing issues, adding documentation, fixing bugs, creating examples, and implementing features.
If you're looking to get started, check out good first issues or issues where help is wanted. For simple documentation changes or typos, feel free to just open a pull request.
If you'd like to contribute, please claim an issue by commenting, forking, and opening a pull request, even if empty. This allows the maintainers to track who is working on what issue as to not overlap work.
Please follow these guidelines:
Before coding:
- choose a branch name that describes the issue you're working on
- enable commit signing
While coding:
- Submit a draft PR asap
- Only change code directly relevant to your PR. Sometimes you might find some code that could really need some refactoring. However, if it's not relevant to your PR, do not touch it. File an issue instead. This allows the reviewer to focus on a single problem at a time.
- If you write comments, do not exceed 80 chars per line. This allows contributors who work with multiple open windows to still read the comments without horizontally scrolling.
- Write adversarial tests. For example, if you're adding a new account type, do not only write tests where the instruction succeeds. Also write tests that test whether the instruction fails, if a check inside the new type is violated.
- Fork the repository and clone it locally.
- Install dependencies:
npm install - Run tests to ensure everything works:
npm test
- Ensure your PR description clearly explains the problem and solution.
- Reference any related issues by number (e.g., "Closes #123").
- Make sure all tests pass before requesting a review.
- Respond to review comments and make necessary changes.
- Follow the existing code style.
- Use Prettier for formatting.
npm run fmt
- Add or update tests for your changes.
- Tests are located in the
core/test/directory. - Run tests with:
npm test
- Please use GitHub Issues to report bugs or request features.
- Include as much detail as possible.