Vanilla-Conquer icon indicating copy to clipboard operation
Vanilla-Conquer copied to clipboard

Ubuntu-based workflows updated to latest versions

Open MLipfert1 opened this issue 3 years ago • 3 comments
trafficstars

Ubuntu 18.04 is deprecated by GitHub. I updated the workflows to the latest Ubuntu-versions and fixed the formattings of some files. Otherwise clang-format-14 would fail.

MLipfert1 avatar Sep 09 '22 20:09 MLipfert1

Can you bump the clang-format version a little less? Say Clang-format12 so its still useable on 20.04 as well which is still supported without having to add 3rd party repos?

OmniBlade avatar Sep 12 '22 14:09 OmniBlade

The clang-format version should not really matter as it is only used in this workflow for checking the source code files for a proper and nice formatting. You should still be able to compile it with lower clang versions, even if the code has been formatted with clang-format-14.

The format adjustments of my second commit would also be required with clang-format-11.

MLipfert1 avatar Sep 12 '22 16:09 MLipfert1

Except that if you are committing code you need to have used matching formatting which is easier if you can use the same version of clang-format the CI uses locally.

OmniBlade avatar Sep 16 '22 22:09 OmniBlade

I set the the clang-format version to "12" for the Linux workflow. I also did a quick fix for the Windows workflow, to install the correct ImageMagick version in any case.

MLipfert1 avatar Oct 02 '22 13:10 MLipfert1