For example, my CI failed yesterday with CodeSpell
Code: Select all
Warning: src/Mod/AddonManager/addonmanager_devmode.py:87: commiters ==> committers
Error: Process completed with exit code 1.
We fixed this in Einstein (https://github.com/pguyot/Einstein) by never committing directly into `master`, but making everything into a Pull Request and wait for CI to finish before merging. Core developers can of course apply their own PR without a peer review, but by waiting for CI to finish first, silly bugs like the typo above are caught before they trigger the master CI or worse, a master that can't be built at all.
- Matthias
PS: happy to explain the detail. GitHub is a huge help here.