cppcheck icon indicating copy to clipboard operation
cppcheck copied to clipboard

fix #13998: Add script for pulling latest simplecpp version

Open ludviggunne opened this issue 5 months ago • 4 comments

ludviggunne avatar Jul 10 '25 08:07 ludviggunne

  • a script just to help you copy 2 files? I think it's pretty quick and easy to do manually.

This way it documents what to do and could be easily run without much copy and paste.

A additional step could be to automatically generate a Cppcheck PR when a new tag is created in the simplecpp repo. Which would leverage the script.

firewave avatar Jul 10 '25 11:07 firewave

This way it documents what to do and could be easily run without much copy and paste.

Documentating the release process etc is valuable. However plain text is less "cryptic". Imho a python script would also be less "cryptic". I.e. instead of head -1 | cut -f2 it is better with corresponding python code.

This way it documents what to do and could be easily run without much copy and paste.

I don't know I still feel I loose a bit of control here. Many things can go wrong if it's used wrongly. Ensure that it can't be used wrongly.

It will always blindly pick latest release... so if I tag a release candidate then the release candidate will be picked.

It would probably be better to commit automatically to enforce that the commit message contains the tag that is used.

And I think some better checking could be good before any files are copied at all.

  • Ensure that current branch is main branch. Not some feature branch.
  • Ensure that main branch is uptodate. No extra commits/changes.
  • => the commit should only contain the expected changes no unrelated changes.
  • whatever other error conditions we can imagine..

danmar avatar Jul 10 '25 12:07 danmar

See https://github.com/danmar/cppcheck/pull/7659 for way to leverage this.

firewave avatar Oct 13 '25 16:10 firewave