[x] I can and do license this contribution under the EFLv2
[x] No issues are reported by make qa (runs make lint and make test)
[x] I have tested the functionality of the things this change touches
Whether the CodeQL workflow runs when I open the PR will be the test, lol
CodeQL workflow doesn't appear on the Checks tab for this PR; seems to have worked 🥳
Notes
No, unnecessary Actions runs don't cost the project anything, but since it's so easy to be nice and not waste even free resources on analyzing non-code changes, I think we might as well.
Thought about it, but can't skip the CI pipeline for unrelated changes because those checks are marked as required. They also finish in just a minute or two, vs. the 4-5 minutes (or longer) that CodeQL takes to run.
Saves us from burning GHA minutes (4-5 min/run) on CodeQL analysis when no files in the
sopel/
directory were changed.Should have done this in #2528 but alas, I forgot.
Checklist
make qa
(runsmake lint
andmake test
)Notes
No, unnecessary Actions runs don't cost the project anything, but since it's so easy to be nice and not waste even free resources on analyzing non-code changes, I think we might as well.
Thought about it, but can't skip the CI pipeline for unrelated changes because those checks are marked as required. They also finish in just a minute or two, vs. the 4-5 minutes (or longer) that CodeQL takes to run.