Check Monorepo Code Styling

adding a linter workflow #2

Sign in to view logs

Triggered via pull request February 5, 2024 15:58

Edwardius

synchronize #61

eddy/add-linting-workflow

Status Success

Total duration 51s

Artifacts –

check_styling.yml

on: pull_request

Run Python linters

run-linters-cpp

Annotations

10 errors, 12 warnings, and 10 notices

src/samples/cpp/producer/src/producer_node.cpp:5:10 [clang-diagnostic-error]: src/samples/cpp/producer/src/producer_node.cpp#L5

'producer_node.hpp' file not found

src/samples/cpp/producer/src/producer_core.cpp:4:10 [clang-diagnostic-error]: src/samples/cpp/producer/src/producer_core.cpp#L4

'producer_core.hpp' file not found

src/samples/cpp/producer/test/producer_test.cpp:1:10 [clang-diagnostic-error]: src/samples/cpp/producer/test/producer_test.cpp#L1

'gtest/gtest.h' file not found

src/samples/cpp/aggregator/src/aggregator_node.cpp:4:10 [clang-diagnostic-error]: src/samples/cpp/aggregator/src/aggregator_node.cpp#L4

'aggregator_node.hpp' file not found

src/samples/cpp/aggregator/src/aggregator_core.cpp:3:10 [clang-diagnostic-error]: src/samples/cpp/aggregator/src/aggregator_core.cpp#L3

'aggregator_core.hpp' file not found

src/samples/cpp/aggregator/test/aggregator_test.cpp:4:10 [clang-diagnostic-error]: src/samples/cpp/aggregator/test/aggregator_test.cpp#L4

'gtest/gtest.h' file not found

src/samples/cpp/transformer/src/transformer_core.cpp:4:10 [clang-diagnostic-error]: src/samples/cpp/transformer/src/transformer_core.cpp#L4

'transformer_core.hpp' file not found

src/samples/cpp/transformer/src/transformer_node.cpp:3:10 [clang-diagnostic-error]: src/samples/cpp/transformer/src/transformer_node.cpp#L3

'transformer_node.hpp' file not found

src/samples/cpp/transformer/test/transformer_test.cpp:5:10 [clang-diagnostic-error]: src/samples/cpp/transformer/test/transformer_test.cpp#L5

'gtest/gtest.h' file not found

src/perception/radar_object_detection/radar_vis/src/radar_vis_node.cpp:3:10 [clang-diagnostic-error]: src/perception/radar_object_detection/radar_vis/src/radar_vis_node.cpp#L3

'radar_vis_node.hpp' file not found

Run Python linters

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, 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/

src/samples/cpp/producer/src/producer_node.cpp:67:5 [modernize-use-trailing-return-type]: src/samples/cpp/producer/src/producer_node.cpp#L67

use a trailing return type for this function

src/samples/cpp/producer/test/producer_test.cpp:5:1 [modernize-use-trailing-return-type]: src/samples/cpp/producer/test/producer_test.cpp#L5

use a trailing return type for this function

src/samples/cpp/producer/test/producer_test.cpp:5:18 [readability-named-parameter]: src/samples/cpp/producer/test/producer_test.cpp#L5

all parameters should be named in a function

src/samples/cpp/producer/test/producer_test.cpp:15:1 [modernize-use-trailing-return-type]: src/samples/cpp/producer/test/producer_test.cpp#L15

use a trailing return type for this function

src/samples/cpp/producer/test/producer_test.cpp:15:18 [readability-named-parameter]: src/samples/cpp/producer/test/producer_test.cpp#L15

all parameters should be named in a function

src/samples/cpp/producer/test/producer_test.cpp:25:1 [modernize-use-trailing-return-type]: src/samples/cpp/producer/test/producer_test.cpp#L25

use a trailing return type for this function

src/samples/cpp/producer/test/producer_test.cpp:25:18 [readability-named-parameter]: src/samples/cpp/producer/test/producer_test.cpp#L25

all parameters should be named in a function

src/samples/cpp/aggregator/src/aggregator_node.cpp:43:5 [modernize-use-trailing-return-type]: src/samples/cpp/aggregator/src/aggregator_node.cpp#L43

use a trailing return type for this function

src/samples/cpp/aggregator/test/aggregator_test.cpp:9:1 [modernize-use-trailing-return-type]: src/samples/cpp/aggregator/test/aggregator_test.cpp#L9

use a trailing return type for this function

src/samples/cpp/aggregator/test/aggregator_test.cpp:9:20 [readability-named-parameter]: src/samples/cpp/aggregator/test/aggregator_test.cpp#L9

all parameters should be named in a function

Run clang-format on src/samples/cpp/producer/src/producer_node.cpp: src/samples/cpp/producer/src/producer_node.cpp#L1

File src/samples/cpp/producer/src/producer_node.cpp does not conform to Custom style guidelines. (lines 7, 8, 10, 11, 13, 14, 28, 31, 32, 35, 40, 41, 48, 49, 55, 56, 57, 60, 67)

Run clang-format on src/samples/cpp/producer/src/producer_core.cpp: src/samples/cpp/producer/src/producer_core.cpp#L1

File src/samples/cpp/producer/src/producer_core.cpp does not conform to Custom style guidelines. (lines 6, 9, 10, 11, 14, 15, 16, 19, 26, 33, 35, 40)

Run clang-format on src/samples/cpp/producer/test/producer_test.cpp: src/samples/cpp/producer/test/producer_test.cpp#L1

File src/samples/cpp/producer/test/producer_test.cpp does not conform to Custom style guidelines. (lines 5, 15, 25)

Run clang-format on src/samples/cpp/aggregator/src/aggregator_node.cpp: src/samples/cpp/aggregator/src/aggregator_node.cpp#L1

File src/samples/cpp/aggregator/src/aggregator_node.cpp does not conform to Custom style guidelines. (lines 6, 7, 8, 9, 10, 11, 13, 14, 15, 16, 18, 19, 20, 21, 25, 26, 29, 30, 34, 35, 38, 39, 43)

Run clang-format on src/samples/cpp/aggregator/src/aggregator_core.cpp: src/samples/cpp/aggregator/src/aggregator_core.cpp#L1

File src/samples/cpp/aggregator/src/aggregator_core.cpp does not conform to Custom style guidelines. (lines 5, 8, 9, 10, 13, 21, 26, 29, 30, 32, 37, 38, 41, 47)

Run clang-format on src/samples/cpp/aggregator/test/aggregator_test.cpp: src/samples/cpp/aggregator/test/aggregator_test.cpp#L1

File src/samples/cpp/aggregator/test/aggregator_test.cpp does not conform to Custom style guidelines. (lines 4, 9, 19, 33, 43, 61)

Run clang-format on src/samples/cpp/transformer/src/transformer_core.cpp: src/samples/cpp/transformer/src/transformer_core.cpp#L1

File src/samples/cpp/transformer/src/transformer_core.cpp does not conform to Custom style guidelines. (lines 6, 9, 12, 17, 18, 19, 22, 23, 28, 36, 37, 38, 44, 45, 52, 56, 57, 63, 67, 68, 74, 82)

Run clang-format on src/samples/cpp/transformer/src/transformer_node.cpp: src/samples/cpp/transformer/src/transformer_node.cpp#L1

File src/samples/cpp/transformer/src/transformer_node.cpp does not conform to Custom style guidelines. (lines 5, 6, 8, 9, 10, 11, 13, 14, 21, 34, 45, 52)

Run clang-format on src/samples/cpp/transformer/test/transformer_test.cpp: src/samples/cpp/transformer/test/transformer_test.cpp#L1

File src/samples/cpp/transformer/test/transformer_test.cpp does not conform to Custom style guidelines. (lines 5, 12, 13, 16, 19, 32, 33, 36, 37, 43, 53, 71, 84, 95, 96, 97, 98, 99, 100, 101, 102)

Run clang-format on src/perception/radar_object_detection/radar_vis/src/radar_vis_node.cpp: src/perception/radar_object_detection/radar_vis/src/radar_vis_node.cpp#L1

File src/perception/radar_object_detection/radar_vis/src/radar_vis_node.cpp does not conform to Custom style guidelines. (lines 3, 6, 7, 9, 10, 11, 12, 13, 14, 17, 18, 21, 25)