GGI Activity: Run Code Reviews
Table of contents
Run code reviews
Link to GitLab issue: https://gitlab.ow2.org/ggi/ggi-castalia/-/issues/44.
Code review is a routine task involving manual and/or automated review of an application’s source code before releasing a product or delivering a project to the customer. In the case of open-source software, code review is more than just about catching errors opportunistically; it is an integrated approach to collaborative development carried out at the team level.
Code reviews should apply to code developed in-house as well as to code reused from external sources, as it improves general confidence in code and reinforces ownership. It is also an excellent way to enhance global skills and knowledge within the team and foster team collaboration.
Code reviews are valuable whenever the organisation develops software or reuses external pieces of software.
While being a standard step in the software engineering process, code reviews in the context of open-source bring specific benefits such as:
- When publishing internal source code, verify adequate quality guidelines are respected.
- When contributing to an existing open source project, verify that guidelines of the targeted project are respected.
- The publicly available documentation is updated accordingly.
It is also an excellent opportunity to share and enforce some of your company legal compliance policy rules, such as:
- Never remove existing licence headers or copyrights found in reused open-source code.
- Do not copy & paste source code from Stack Overflow without prior permission from the legal team.
- Include the correct copyright line when required.
Code reviews will bring trust and confidence to code. If people are not sure about the quality or potential risks of using a software product, they should conduct peer- and code- reviews.
The following verification points demonstrate progress in this Activity:
- Code review is a collective task that works better in a good collaborative environment.
- Do not hesitate to use existing tools and patterns from the open-source world, where code reviews have been a standard for years (decades).