geany-plugins icon indicating copy to clipboard operation
geany-plugins copied to clipboard

CI: Download, compile and use latest cppcheck version

Open eht16 opened this issue 4 months ago • 2 comments

Instead of using a dated version of "cppcheck" from the runner image, we download the latest release of "cppcheck" and build it.

As building it takes a few minutes, the build result is cached to be re-used in succeeding build runs.

This should make us aware of failed checks in recent "cppcheck" versions quicker. Currently we notice these mostly only when running "make check" locally or when the nightly builds for Debian Unstable fail because they also use recent "cppcheck" versions.

Currently the build fails because of the pending changes in #1309, once this is merged, the build should succeed.

eht16 avatar Feb 17 '24 11:02 eht16

Have you ever considered adding a CodeQL check here or on Geany itself?

andy5995 avatar Feb 17 '24 13:02 andy5995

Have you ever considered adding a CodeQL check here or on Geany itself?

Not relevant to this PR, if "somebody" ran it on the source and posted the results in a separate issue it might be interesting, but please do not pollute PRs with other topics.

elextr avatar Feb 17 '24 13:02 elextr

If there are no objections or improvement suggestions, I would like to merge to this in a week.

eht16 avatar Mar 28 '24 08:03 eht16

If it's working -- why not. I have absolutely no experience with this part of the universe and therefor no objective opinion.

frlan avatar Apr 04 '24 20:04 frlan