search

NextPP / sudoManager

Restaurant manager that you would love to hire.
MIT License
3 stars 6 forks source link

Added TableSeat class and Table class #31

Closed derekjtong closed 1 year ago

derekjtong commented 1 year ago

closes #14

derekjtong commented 1 year ago

Still failing checks, I'm not sure what is wrong

neg-c commented 1 year ago

Use this on the root of the project to pass Check_Code_Format find . -regex '.*\.\(cpp\|hpp\|cxx\|cp\|cc\|cu\|c\|h\)' -not -path '*/extern/*' -exec clang-format -style=file -i {} \; and this to pass Check_Cmake_Format find . \( -name '*.cmake' -o -name 'CMakeLists.txt' \) -not -path '*/extern/*/*' -exec cmake-format -i {} \;

Committing the changes produced with these commands should do it.

derekjtong commented 1 year ago

find . -regex '.*\.\(cpp\|hpp\|cxx\|cp\|cc\|cu\|c\|h\)' -not -path '*/extern/*' -exec clang-format -style=file -i {} \;

I ran this in the project root. No change were made, however it is still failing the check. @kelarov would it be possible for you to try it please? Thank you.

Also thank you for the help @genciberisha . I found where the commands are (PR-main.yml), now I know where to look in the future.

derekjtong commented 1 year ago

Update-I switched to different OS (macOS to Ubuntu) and the command made some changes.

neg-c commented 1 year ago

Update-I switched to different OS (macOS to Ubuntu) and the command made some changes.

You shouldn't switch OS-s for this. Can you tell me your setup for future reference?

derekjtong commented 1 year ago

Sure, what details about my setup specifically? I'm not sure what to put, here's some stuff to start