As discussed offline, just making `yarn test` do what we expect it to. ### Change Type - [x] `internal` — Any other changes that don't affect the published package[^2]
1.9 KiB
Contributing
Thank you for your interest in contributing to tldraw! We welcome any contributions to the code base and the documentation.
Create an Issue!
Before submitting a pull request, it is strongly recommended to create an issue first to discuss your proposed changes. This will help us to make sure that your changes are aligned with the project goals and that you are not duplicating work that is already in progress.
If you are not sure whether your changes are needed, feel free to create an issue anyway and we can discuss it there. Once we have agreed on the changes, you can start working on them.
Making Changes
To create a pull request:
- Fork and clone the repository
- Create a separate branch for your changes
- Make your changes, and ensure that it is formatted by Prettier and type-checks without errors in TypeScript
- Write tests that validate your change and/or fix.
- Run
yarn build
and then run tests withyarn test-ci
. - Push your branch and open a PR. 🚀
Before your code is merged, you will need to sign our contributor license agreement, which is handled automatically via GitHub comments. Your PR will be reviewed and merged in within a day or two if everything looks good.
Please also see our Code of Conduct for our expectations around contributor culture.