DoozyX / clang-format-lint-action

This action checks if the source code matches the .clang-format file.
MIT License
129 stars 46 forks source link

Add clang-format 17.0.2 #60

Closed alemuntoni closed 6 months ago

offa commented 7 months ago

@DoozyX could you have a look at this PR? Latest clang-format would be appreciated :-).

alemuntoni commented 7 months ago

Ooops! Sorry, didn't see that! I just fixed it.

DoozyX commented 7 months ago

thanks, but still don't think it is working as expected, are you testing it on unix, this is what I got

image
DoozyX commented 7 months ago

also where does the binary come from, for consistency can you align it with https://github.com/muttleyxd/clang-tools-static-binaries?

alemuntoni commented 7 months ago

also where does the binary come from

I downloaded it from the official llvm release on github, since I did not find anywhere from where they came from.

I will align the binary from the link you provided.

DoozyX commented 7 months ago

thanks, in that case just posting the link should be fine

alemuntoni commented 7 months ago

I replaced the binary with the one provided in the https://github.com/muttleyxd/clang-tools-static-binaries repo, which is version 17.0.4.

DoozyX commented 7 months ago

Also the binary needs to be executable (needs chmod +x)

alemuntoni commented 7 months ago

Also the binary needs to be executable (needs chmod +x)

Ops again :D Sorry!

It should be ok now!

GeckoEidechse commented 6 months ago

Closes #64

DoozyX commented 6 months ago

Thanks