search

StoglRobotics / ros_team_workspace

RosTeamWorkspace is a framework for boosting collaboration in teams when developing software for robots using Robot Operating System (ROS1 and ROS2).
https://rtw.stoglrobotics.de
Apache License 2.0
79 stars 17 forks source link

Add separate script for setting up formatting for a repository. #153

Closed destogl closed 8 months ago

destogl commented 9 months ago

@SahWag can you test this changes. The best test is to do it in the wild. My proposal:

  1. Setup formatting for all repositories that are stored in GitLab for the current project (use setup-formatting alias)
  2. Commit the formatting setup and after that format the code as instructed at the end of the script and commit this separately.
  3. Setup CI on the open-source repositories used in current project using ṡetup-repository-ci alias. This should also automatically setup the formatting.
  4. After setting the first repository, compare the CI setup to ros2_control and report any differences. We probably have to update our templates.
  5. Merge / push configuration and changes separately. Then you will also see if there are any changes on the CI setup needed.
  6. Report any issues or needed changes.
  7. Finally, document the functionality in this repository, if something is missing.
SahWag commented 9 months ago

I've added a pull request for the only significant issue I've found: https://github.com/StoglRobotics/ros_team_workspace/pull/154 Apart from that, there's no direct issues in this PR, but more documentation on CI and how RTW sets up CI might be nice. I'll try and write smoe documentation on this in another PR