Enable CodeQL scanning ⚠️

[JamieSlome]

Tasks

Beta Give feedback

1. Enable CodeQL (Extended)
2. Ensure CodeQL issues are resolved before merging into main

I'd like us to have a baseline tool in place and we can have future discussions about rigidifying the PR review process. @maoo, I have access to some of the repository settings but am not able to enable CodeQL. I believe this is only possible at the organization level or via administrative controls.

Thank you! 🎉

[JamieSlome]

I just remembered that this can also be enabled via the creation of a configuration file too 👍 The administrative "switch" is easier, but creating the config file will work too...

[JamieSlome]

@maoo - I'll take it on and just open a PR for you to review 👍

(cc) @coopernetes

[JamieSlome]

@maoo - I've opened up #403 but will need you to "switch-on" the requirement for passing status check before PRs can be merged into main.

[maoo]

@JamieSlome - I've enabled CodeQL analysis , I see the CodeQL check that passed on #403 - please let me know if there's any tweaking to make.

[JamieSlome]

@maoo - thank you! Can you make the status check required, i.e. it has to pass before the merge can occur?

BTW, could you also make the other status checks required as well?

[maoo]

Done; since I was there, I made few more additions that IMO made a lot of sense; please let me know if I jumped the gun, I'm happy to revert the changes 😄

• Require conversation resolution before merging
• Require branches to be up to date before merging
• Status checks that are required - CodeQL (new) and EasyCLA

[JamieSlome]

Thanks! The question is, why is #403 now failing? 🤔

[maoo]

Thanks! The question is, why is #403 now failing? 🤔

https://www.returngis.net/2023/05/como-solucionar-el-error-codeql-analyses-from-advanced-configurations-cannot-be-processed-when-the-default-setup-is-enabled-en-ghas/

Seems that specifying the CodeQL configuration via a GitHub Action addresses the issue; I'll raise a PR

[maoo]

#406 - green! Feel free to review and merge

[JamieSlome]

Addressed in #406.

@maoo ❤️