search

rat-pac / ratpac-two

Open source edition of RAT-PAC
Other
11 stars 23 forks source link

Catch case where clang-format is not found/installed #108

Closed masmiley closed 9 months ago

masmiley commented 10 months ago

The new autoformatting feature assumes that clang-format will be present on the system, when that's not necessarily the case. We should add something to catch that case instead of seeing the generic command not found error message. The message does not prevent the formatting script from completing and just means it will do nothing over than print the message to the screen for each file, so it is not particularly urgent and could wait until a new round of build-related revisions.

tannerbk commented 9 months ago

Should be fixed, can you confirm?

JamesJieranShen commented 9 months ago

I believe this is completed. Feel free to re-open if needed.