-
Notifications
You must be signed in to change notification settings - Fork 1
Branches
Mikhail Deriabin edited this page Aug 27, 2024
·
1 revision
The development process should follow these rules:
- The dev branch is the first branch, where all PRs should go. Anybody can make a PR to this branch. All PRs and merges from this branch are going to be done to the main branch only.
- The main branch should contain stable code only. Any small changes should happen in the dev branch. Only repository admins can make PRs and merges to this branch. All PRs and merges from this branch are going to be done to the prod branch only.
- The prod branch should contain only stable and well-tested code since code from this branch is going to be deployed to the real server. No PRs or merges are allowed from this branch.
- All repository collaborators are responsible for the stability of the dev branch and should check that their own changes are not breaking the existing code, at least in their own branches from which PRs to the dev branch are coming.
- The repository admin is responsible for reviewing all PRs coming to the dev branch. The admin is also responsible for syncing the dev, main, and prod branches and verifying their stability.
Branches are usually created one for one issue and because of that they usually should not be in use for a long time and removed after PR to the dev branch was accepted and merged.
Contributors should follow these conventions in branch naming:
- At the first place of branch goes an issue type with "/" in the end
- After that some short free form descriptive name, kebab-case
- In the end there should be a number of the issue to which branch is connected to
- feature - new feature to be added (all issues with feature label)
- change - change of existing functionality (all issues with change label)
- bug - fixing some bug (all issues with bug label)
- docs - updating some docs (JSDocs, line comments, swagger etc.) (all issues with docs label)
-
feature/authorization-clan-45
new feature for adding authorization rules, associated issue number 45 -
docs/swagger-stock-20
add documentation for /stock endpoint, associated issue number 20