Split up classes in planner_node into 3 files #1623
petergarud
This run and associated checks have been archived and are scheduled for deletion.
Learn more about checks retention
ci.yml
on: pull_request
build-and-test
21m 20s
fix-style
1m 25s
Annotations
19 warnings
|
fix-style
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
|
fix-style
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
build-and-test
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
|
build-and-test
'RRT::GridStateSpace::intermediateState' hides overloaded virtual function
|
|
build-and-test
unused variable 'n'
|
|
build-and-test
field '_kdtree' will be initialized after field '_dimensions'
|
|
build-and-test
field '_dimensions' will be initialized after field '_nodemap'
|
|
build-and-test
field '_parent' will be initialized after field '_state'
|
|
build-and-test
field '_state' will be initialized after field '_vec'
|
|
build-and-test
'RRT::GridStateSpace::intermediateState' hides overloaded virtual function
|
|
build-and-test
unused variable 'n'
|
|
build-and-test
'mousePressEvent' overrides a member function but is not marked 'override'
|
|
build-and-test
'mouseMoveEvent' overrides a member function but is not marked 'override'
|
|
build-and-test:
soccer/src/soccer/planning/global_state.hpp#L66
return type 'const rj_msgs::msg::CoachState' (aka 'const CoachState_<std::allocator<void>>') is 'const'-qualified at the top level, which may reduce code readability without improving const correctness
|
|
build-and-test:
soccer/src/soccer/planning/planner_for_robot.cpp#L53
the const qualified parameter 'request' is copied for each invocation; consider making it a reference
|
|
build-and-test:
soccer/src/soccer/planning/planner_for_robot.cpp#L93
method 'get_time_left' can be made static
|
|
build-and-test:
soccer/src/soccer/planning/planner_for_robot.cpp#L113
Value stored to 'play_state' during its initialization is never read
|
|
build-and-test:
soccer/src/soccer/planning/planner_for_robot.cpp#L222
catch handler catches by value; should catch by reference instead
|
|
build-and-test:
soccer/src/soccer/planning/planner_for_robot.cpp#L245
missing username/bug in TODO
|