Skip to content

cpp and python linters #21

cpp and python linters

cpp and python linters #21

Triggered via pull request February 5, 2024 18:02
Status Success
Total duration 30s
Artifacts

check_styling.yml

on: pull_request
Run Python linters
9s
Run Python linters
Run C++ linters
20s
Run C++ linters
Fit to window
Zoom out
Zoom in

Annotations

56 errors and 3 warnings
src/samples/python/producer/setup.py#L10
packages=[package_name], data_files=[ # Install marker file in the package index - ('share/ament_index/resource_index/packages', ['resource/' + package_name]), + ('share/ament_index/resource_index/packages', + ['resource/' + package_name]), # Include our package.xml file (os.path.join('share', package_name), ['package.xml']), # Include all launch files (os.path.join('share', package_name, 'launch'), glob(os.path.join('launch', '*.launch.py'))), # Include config files for parameters - (os.path.join('share', package_name, 'config'), glob(os.path.join('config', '*.yaml'))), + (os.path.join('share', package_name, 'config'), + glob(os.path.join('config', '*.yaml'))), ], install_requires=['setuptools'], zip_safe=True,
src/samples/python/producer/producer/producer_node.py#L36
pos_x = self.get_parameter('pos_x').get_parameter_value().double_value pos_y = self.get_parameter('pos_y').get_parameter_value().double_value pos_z = self.get_parameter('pos_z').get_parameter_value().double_value - velocity = self.get_parameter('velocity').get_parameter_value().double_value + velocity = self.get_parameter( + 'velocity').get_parameter_value().double_value # Initialize producer core logic for serialization self.__producer = ProducerCore(pos_x, pos_y, pos_z, velocity) # Initialize ROS2 constructs queue_size = 10 - self.publisher_ = self.create_publisher(Unfiltered, '/unfiltered_topic', queue_size) + self.publisher_ = self.create_publisher( + Unfiltered, '/unfiltered_topic', queue_size) timer_period = 0.5 self.timer = self.create_timer(timer_period, self.__publish_position)
src/samples/python/aggregator/setup.py#L10
packages=[package_name], data_files=[ # Install marker file in the package index - ('share/ament_index/resource_index/packages', ['resource/' + package_name]), + ('share/ament_index/resource_index/packages', + ['resource/' + package_name]), # Include our package.xml file (os.path.join('share', package_name), ['package.xml']), # Include all launch files.
src/samples/python/aggregator/aggregator/aggregator_node.py#L54
str(self.__aggregator.num_filtered_msgs)) self.get_logger().info('Producer Frequency:' + str(self.__aggregator.raw_freq)) - self.get_logger().info('Transformer Frequency:' + str(self.__aggregator.filtered_freq)) + self.get_logger().info('Transformer Frequency:' + + str(self.__aggregator.filtered_freq)) def main(args=None):
src/samples/python/transformer/setup.py#L10
packages=[package_name], data_files=[ # Install marker file in the package index - ('share/ament_index/resource_index/packages', ['resource/' + package_name]), + ('share/ament_index/resource_index/packages', + ['resource/' + package_name]), # Include our package.xml file (os.path.join('share', package_name), ['package.xml']), # Include all launch files. (os.path.join('share', package_name, 'launch'), glob(os.path.join('launch', '*.launch.py'))), # Include config files for parameters - (os.path.join('share', package_name, 'config'), glob(os.path.join('config', '*.yaml'))), + (os.path.join('share', package_name, 'config'), + glob(os.path.join('config', '*.yaml'))), ], install_requires=['setuptools'], zip_safe=True,
src/samples/python/transformer/transformer/transformer_node.py#L35
self.__transformer = TransformerCore() # Initialize ROS2 Constructs - self.publisher_ = self.create_publisher(FilteredArray, '/filtered_topic', 10) + self.publisher_ = self.create_publisher( + FilteredArray, '/filtered_topic', 10) self.subscription = self.create_subscription(Unfiltered, '/unfiltered_topic', self.unfiltered_callback, 10)
src/perception/radar_object_detection/radar_vis/include/radar_vis_node.hpp#L15
code should be clang-formatted [-Wclang-format-violations]
src/perception/radar_object_detection/radar_vis/include/radar_vis_node.hpp#L28
code should be clang-formatted [-Wclang-format-violations]
src/perception/radar_object_detection/radar_vis/include/radar_vis_node.hpp#L43
code should be clang-formatted [-Wclang-format-violations]
src/perception/radar_object_detection/radar_vis/include/radar_vis.hpp#L14
code should be clang-formatted [-Wclang-format-violations]
src/perception/radar_object_detection/radar_vis/include/radar_vis.hpp#L22
code should be clang-formatted [-Wclang-format-violations]
src/perception/radar_object_detection/radar_vis/include/radar_vis.hpp#L24
code should be clang-formatted [-Wclang-format-violations]
src/perception/radar_object_detection/radar_vis/src/radar_vis_node.cpp#L24
code should be clang-formatted [-Wclang-format-violations]
src/perception/radar_object_detection/radar_vis/src/radar_vis_node.cpp#L24
code should be clang-formatted [-Wclang-format-violations]
src/perception/radar_object_detection/radar_vis/src/radar_vis.cpp#L83
code should be clang-formatted [-Wclang-format-violations]
src/perception/radar_object_detection/radar_vis/test/radar_vis_test.cpp#L6
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/include/aggregator_core.hpp#L6
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/include/aggregator_core.hpp#L9
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/include/aggregator_core.hpp#L16
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/include/aggregator_core.hpp#L47
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/include/aggregator_core.hpp#L55
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/include/aggregator_core.hpp#L73
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/include/aggregator_core.hpp#L75
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/include/aggregator_node.hpp#L6
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/include/aggregator_node.hpp#L15
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/include/aggregator_node.hpp#L29
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/include/aggregator_node.hpp#L29
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/include/aggregator_node.hpp#L30
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/include/aggregator_node.hpp#L34
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/include/aggregator_node.hpp#L39
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/include/aggregator_node.hpp#L43
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/include/aggregator_node.hpp#L50
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/include/aggregator_node.hpp#L56
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/src/aggregator_core.cpp#L5
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/src/aggregator_core.cpp#L8
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/src/aggregator_core.cpp#L9
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/src/aggregator_core.cpp#L10
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/src/aggregator_core.cpp#L13
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/src/aggregator_core.cpp#L21
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/src/aggregator_core.cpp#L26
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/src/aggregator_core.cpp#L29
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/src/aggregator_core.cpp#L30
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/src/aggregator_core.cpp#L32
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/src/aggregator_core.cpp#L32
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/src/aggregator_core.cpp#L37
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/src/aggregator_core.cpp#L38
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/src/aggregator_core.cpp#L41
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/src/aggregator_core.cpp#L47
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/src/aggregator_node.cpp#L6
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/src/aggregator_node.cpp#L7
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/src/aggregator_node.cpp#L8
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/src/aggregator_node.cpp#L9
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/src/aggregator_node.cpp#L10
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/src/aggregator_node.cpp#L11
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/src/aggregator_node.cpp#L11
code should be clang-formatted [-Wclang-format-violations]
src/samples/cpp/aggregator/src/aggregator_node.cpp#L13
code should be clang-formatted [-Wclang-format-violations]
Run Python linters
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-python@v1, wearerequired/lint-action@v2. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Run Python linters
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-python@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
Run C++ linters
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: wearerequired/lint-action@v2. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.