Thanks for your interest in contributing to @webtides/layouts
! Please take a moment to review this document before submitting a pull request.
Please ask first before starting work on any significant new features.
It's never a fun experience to have your pull request declined after investing a lot of time and effort into a new feature. To avoid this from happening, we request that contributors create an issue to first discuss any significant new features.
We use ESLint and Prettier to ensure good code quality.
ESLint & Prettier will run automatically when staging files via git
.
You can run the test suite using the following commands:
npm test
Please ensure that the tests are passing when submitting a pull request. If you're adding new features to @webtides/layouts
, please include tests.
We use a trunk-based development workflow.
In the trunk-based development model, all developers work on a single branch with open access to it. Often it’s simply the
main
branch. They commit code to it and run it. It’s super simple. In some cases, they create short-lived feature branches. Once code on their branch compiles and passes all tests, they merge it straight tomain
. It ensures that development is truly continuous and prevents developers from creating merge conflicts that are difficult to resolve.
As a Release is complete the main
branch will be tagged with the new release version.
Pull requests should take place whenever a:
- FEATURE is about to be finished
- RELEASE is about to be finished
When all Reviewers approved a PR the feature/release may be finished locally and pushed to the remote