Project Contribution
Gitlab flow
- If you aren't familiar with gitlab flow, please take a look. We adhere strictly to the gitlab flow and it essential to get learn it.
- Always follow [/development-experience/release-process](Release process guidelines)
- Read no breaking change policy
- TODO: Link to the Release powerhouse
Working with service
- If you are working with a service for the first time, understand its depenencies and make sure you can run tests locally
- Every service has an owner, its a person who can help if you'll have any questions or problems with a service
- Always, dont forget to pull
masterbranch before working with service - Take a look on open PRs to understand if anyone else is also currently working with a service
- Branch agains
masterand open PRs intomaster - Run tests after merging to
master
For the owners
- If you are an owner, don't forget to keep readme, swagger and service description updated
- TODO: create a good example of the service
Git guidelines
- Work in your branch and keep your git history flat
- Separate subject from body with a blank line,
git shortlog, must how subject line for concision:
$ git shortlog
Kevin Flynn (1):
Derezz the master control program
Alan Bradley (1):
Introduce security program "Tron"
Ed Dillinger (3):
Rename chess program to "MCP"
Modify chess program
Upgrade chess program
Walter Gibbs (1):
Introduce protoype chess programGeneral recommendations
- Commit early, commit often
- Read the documentation (and then read it again)
- Optimize CI stages, failures are easy to identify and fix
- Make builds fast and simple
- Use failures to improve processes. Focus on cultural shift for continuous improvement
- Test environment should mirror production
- Wanna read more? follow gitlab blog!
Workflow
- Read, understand the task.
- Pull master changes, create new branch using git branch guidelines.
- Start working with task by creating tests and thinking through the whole solution.
- Implement your code.
- Make sure your tests are working.
- Rebase your branch agains
masterbefore you open PR - Open PR to the
masterbranch and wait for the comments/approvement from other engineers - Move your ticket and PR to the review column
- Address comments from your teammates
- Merge your PR once it approved
- Test your changes on development environment to make sure its properly working
- TODO: Move your ticket to the dome column??