search

eteran / c-vector

A dynamic array implementation in C similar to the one found in standard C++
MIT License
737 stars 109 forks source link

Let CI check the format #29

Closed marekmosna closed 2 years ago

marekmosna commented 2 years ago

When you added the formatter config, you maybe will appreciate robo-check. It's always nice when CI plays the music for free.

eteran commented 2 years ago

Ironically, the PR somehow failed the format check, LOL. My guess is that they are using a slightly different version of clang-format that perhaps has some new settings which are defaulted? I'll look into it, but I like the concept.

marekmosna commented 2 years ago

Weird. When I deployed it the behavior was the same in CI as well as locally invoked with -i parameter

If your repository has a .clang-format file in the root directory then you can use file option here.

for me it's doing always the same change. Reorder includes alphabetically.

eteran commented 2 years ago

OK, I just ran clang-format locally again and it had a minor change it wanted to apply. Could you merge master into your branch to see if it passes now?