Recommended way to format python code?

RFRIEDM · November 28, 2023, 7:13pm

I’m using ament_black in pre-commit. It works great.

Here’s the PR to add support.

github.com/ArduPilot/ardupilot

DDS use black format

ArduPilot:master ← Ryanf55:dds-use-black-format

opened 02:49AM - 22 Nov 23 UTC

Ryanf55

+46 -117

# Purpose Switch to black for formatting. I broke the automatic format into a… single commit. After this PR, the colcon tests pass locally for all the ROS 2 stuff in ArduPilot. # Testing ``` colcon build --packages-up-to ardupilot_dds_tests colcon test --packages-up-to ardupilot_dds_tests colcon test-result --all ``` OR ``` colcon test --packages-up-to ardupilot_dds_tests --event-handlers=console_cohesion+ ``` ``` Label Time Summary: black = 0.22 sec*proc (1 test) copyright = 0.23 sec*proc (1 test) lint_cmake = 0.08 sec*proc (1 test) linter = 0.90 sec*proc (5 tests) pep257 = 0.12 sec*proc (1 test) xmllint = 0.25 sec*proc (1 test) Total Test time (real) = 0.90 sec --- Finished <<< ardupilot_sitl [0.97s] ``` # Known issues * Seems like colcon test-result doesn't publish results for all packages. See https://github.com/botsandus/ament_black/issues * ardupilot_dds_tests is ament_pytest type, and there is no documented way to enforce style in that kind of package. See https://github.com/Timple/ament_black/issues/5

Topic		Replies	Views
python black formatting Next Generation ROS ros2 , ci	27	3493	March 22, 2021
Inconsistency in flake8 between ROS2 (source builds + ci.ros2.org) and (build.ros2.org for PRs)? Next Generation ROS	4	1180	October 18, 2019
Consideration of STEP or Collada for Describing the ROS Environment ROS-Industrial	5	1468	December 9, 2019
MicroXRCE DDS on Ardupilot Embedded ros2	12	7942	April 2, 2023
ROS2 Logging Format Next Generation ROS	9	4003	November 22, 2019

Recommended way to format python code?

Related topics