Addition of code formatting check in CI

Autonomous-Motorsports-Purdue / AMP_ASSv2

Reassembled, simpler, without move_base and loop closure issues

5 stars 1 forks source link

Addition of code formatting check in CI #18

Closed alanssitis closed 2 years ago

alanssitis commented 2 years ago

Description

Implement a CI action checking whether code base is formatted according to specifications to be determined

Sub-tasks

-[] TBD

Resources

https://moveit.ros.org/documentation/contributing/code/

RishiKumarRay commented 2 years ago

which formatting tool you want use?

alanssitis commented 2 years ago

Hi @RishiKumarRay, at this point we don't know yet. I have only done skimming on possible auto-formaters. I haven't found one for ROS projects yet, and if such doesn't exist I think the best path forward is using language-specific formatters (e.g. autopep8 for python and clang-format). Right now this issue is still being researched and developed but all recommendations are welcome!

alanssitis commented 2 years ago

By the way, are you a member of AMP?

RishiKumarRay commented 2 years ago

@alanssitis No, I am not , whats that?

zghera commented 2 years ago

@RishiKumarRay Autonomous Motorsports Purdue. We are an organization at Purdue University. At this time, we are only accepting contributions from members of the club.