Remove trailing whitespace

[leander-dsouza]

Description

I have added the following hooks as part of the pre-commit:

• end-of-file-fixer
• mixed-line-ending
• trailing-whitespace

This prevents empty whitespace and additional newlines from registering as a separate commit during development.

[fujitatomoya]

please see my comment on ros2/ros2_documentation#5203 (review)

[christophebedard]

This is almost certainly going to change existing files, right? Also, note that, given that we have a ton of repos, it would take a non-trivial amount of work to maintain this.

[fujitatomoya]

if we enable this globally, yeah we need to to make sure all repos to pass the github action with pre-commit, that is the work. (my thought is that keep the github action under ros2/.github repository as implementation, and use it to each repo one by one gradually to add the workflow file. so it will not be giant global switch ON/OFF i think.) basically what pre-commit does is right things, we can see that from this PR.

[nuclearsandwich]

@ros-pull-request-builder retest this please

[nuclearsandwich]

@ros-pull-request-builder retest this please

[leander-dsouza]

Should I close this PR and open a new one in the ros2 repository, targeting each package individually?

[fujitatomoya]

@leander-dsouza how would you want to process this? i think we can remove pre-commit thing from here, and merge the rest? Or if you want to close this, that is also fine.

[leander-dsouza]

@fujitatomoya I have refactored the PR to remove the trailing whitespace :)

[fujitatomoya]

Pulls: #2791
Gist: https://gist.githubusercontent.com/fujitatomoya/2e3e7616c2133ca52f1a21f75fdb3b63/raw/67286d237758d2aa3c2ee4a33c48a192e7cfc509/ros2.repos
BUILD args: --packages-up-to rclcpp rclcpp_components
TEST args: --packages-select rclcpp rclcpp_components
ROS Distro: rolling
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/15577

• Linux
• Linux-aarch64
• Linux-rhel
• Windows