This guide outlines the process for submitting Pull Requests and ensuring code quality in our project.
When you submit a Pull Request, our Continuous Integration (CI) system will run several checks automatically. To avoid surprises and streamline the review process, we strongly recommend running these checks locally before submitting your PR.
- Podman must be installed on your system.
Run the following command in your terminal:
./development/code_quality.sh
This script will perform all necessary quality checks on your code.
Our quality assurance process includes the following checks:
We use megalinter to perform various linting tasks:
- BASH script linting
- Markdown linting
- YAML linting
- GitHub Action linting
- Repository secret scanning (using GitLeaks and Credentials scan)
We use REUSE to ensure proper license compliance. This tool checks that every file has valid copyright information in the SPDX-standard format.
Conform is used to check commit messages against our project's commit guidelines. For more details, refer to the CONTRIBUTING.md
file.
After running the quality checks, you'll receive a summary report indicating whether all checks passed or if any failed.
- Review the terminal output for specific error messages.
- Fix the identified issues in your code.
- Stage your changes (
git add
). - Re-run the quality check script.
- Repeat until all checks pass.
- Run the code quality script locally to reproduce the CI errors.
- Fix the issues in your local environment.
- Update your Pull Request with the fixes.
- The CI tests should now pass in the updated PR.
Visual examples of the quality check outputs: