Skip to content

Commit 119d08c

Browse files
github-actions[bot]christophfroehlichsaikishor
authored
Bump version of pre-commit hooks (#2156)
Co-authored-by: christophfroehlich <[email protected]> Co-authored-by: Sai Kishor Kothakota <[email protected]>
1 parent b66e5c6 commit 119d08c

32 files changed

+381
-266
lines changed

‎.pre-commit-config.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -56,14 +56,14 @@ repos:
5656
args: ["--line-length=99"]
5757

5858
- repo: https://github.com/pycqa/flake8
59-
rev: 7.1.2
59+
rev: 7.2.0
6060
hooks:
6161
- id: flake8
6262
args: ["--extend-ignore=E501"]
6363

6464
# CPP hooks
6565
- repo: https://github.com/pre-commit/mirrors-clang-format
66-
rev: v19.1.7
66+
rev: v20.1.0
6767
hooks:
6868
- id: clang-format
6969
args: ['-fallback-style=none', '-i']
@@ -133,7 +133,7 @@ repos:
133133
exclude: CHANGELOG\.rst|\.(svg|pyc|drawio)$
134134

135135
- repo: https://github.com/python-jsonschema/check-jsonschema
136-
rev: 0.31.2
136+
rev: 0.32.1
137137
hooks:
138138
- id: check-github-workflows
139139
args: ["--verbose"]

‎controller_interface/src/chainable_controller_interface.cpp

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -247,8 +247,9 @@ ChainableControllerInterface::on_export_reference_interfaces()
247247
std::vector<hardware_interface::CommandInterface> reference_interfaces;
248248
for (size_t i = 0; i < exported_reference_interface_names_.size(); ++i)
249249
{
250-
reference_interfaces.emplace_back(hardware_interface::CommandInterface(
251-
get_node()->get_name(), exported_reference_interface_names_[i], &reference_interfaces_[i]));
250+
reference_interfaces.emplace_back(
251+
hardware_interface::CommandInterface(
252+
get_node()->get_name(), exported_reference_interface_names_[i], &reference_interfaces_[i]));
252253
}
253254
return reference_interfaces;
254255
}

‎controller_interface/test/test_chainable_controller_interface.hpp

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -71,8 +71,9 @@ class TestableChainableControllerInterface
7171
{
7272
std::vector<hardware_interface::StateInterface> state_interfaces;
7373

74-
state_interfaces.push_back(hardware_interface::StateInterface(
75-
name_prefix_of_interfaces_, "test_state", &state_interfaces_values_[0]));
74+
state_interfaces.push_back(
75+
hardware_interface::StateInterface(
76+
name_prefix_of_interfaces_, "test_state", &state_interfaces_values_[0]));
7677

7778
return state_interfaces;
7879
}
@@ -82,8 +83,9 @@ class TestableChainableControllerInterface
8283
{
8384
std::vector<hardware_interface::CommandInterface> command_interfaces;
8485

85-
command_interfaces.push_back(hardware_interface::CommandInterface(
86-
name_prefix_of_interfaces_, "test_itf", &reference_interfaces_[0]));
86+
command_interfaces.push_back(
87+
hardware_interface::CommandInterface(
88+
name_prefix_of_interfaces_, "test_itf", &reference_interfaces_[0]));
8789

8890
return command_interfaces;
8991
}

‎controller_interface/test/test_force_torque_sensor.cpp

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -39,9 +39,10 @@ TEST_F(ForceTorqueSensorTest, validate_all_with_default_names)
3939
ASSERT_EQ(force_torque_sensor_->state_interfaces_.capacity(), size_);
4040

4141
// validate the default interface_names_
42-
ASSERT_TRUE(std::equal(
43-
force_torque_sensor_->interface_names_.begin(), force_torque_sensor_->interface_names_.end(),
44-
full_interface_names_.begin(), full_interface_names_.end()));
42+
ASSERT_TRUE(
43+
std::equal(
44+
force_torque_sensor_->interface_names_.begin(), force_torque_sensor_->interface_names_.end(),
45+
full_interface_names_.begin(), full_interface_names_.end()));
4546

4647
// get the interface names
4748
std::vector<std::string> interface_names = force_torque_sensor_->get_state_interface_names();

‎controller_interface/test/test_imu_sensor.cpp

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -38,9 +38,10 @@ TEST_F(IMUSensorTest, validate_all)
3838
ASSERT_EQ(imu_sensor_->state_interfaces_.capacity(), size_);
3939

4040
// validate the default interface_names_
41-
ASSERT_TRUE(std::equal(
42-
imu_sensor_->interface_names_.begin(), imu_sensor_->interface_names_.end(),
43-
full_interface_names_.begin(), full_interface_names_.end()));
41+
ASSERT_TRUE(
42+
std::equal(
43+
imu_sensor_->interface_names_.begin(), imu_sensor_->interface_names_.end(),
44+
full_interface_names_.begin(), full_interface_names_.end()));
4445

4546
// get the interface names
4647
std::vector<std::string> interface_names = imu_sensor_->get_state_interface_names();

‎controller_interface/test/test_led_rgb_device.cpp

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -38,9 +38,10 @@ TEST_F(LedDeviceTest, validate_all)
3838
ASSERT_EQ(led_device_->command_interfaces_.capacity(), size_);
3939

4040
// Validate default interface_names_
41-
EXPECT_TRUE(std::equal(
42-
led_device_->interface_names_.cbegin(), led_device_->interface_names_.cend(),
43-
full_cmd_interface_names_.cbegin(), full_cmd_interface_names_.cend()));
41+
EXPECT_TRUE(
42+
std::equal(
43+
led_device_->interface_names_.cbegin(), led_device_->interface_names_.cend(),
44+
full_cmd_interface_names_.cbegin(), full_cmd_interface_names_.cend()));
4445

4546
// Get interface names
4647
std::vector<std::string> interface_names = led_device_->get_command_interface_names();

‎controller_interface/test/test_pose_sensor.cpp

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -36,9 +36,10 @@ TEST_F(PoseSensorTest, validate_all)
3636
ASSERT_EQ(pose_sensor_->state_interfaces_.capacity(), size_);
3737

3838
// Validate default interface_names_
39-
EXPECT_TRUE(std::equal(
40-
pose_sensor_->interface_names_.cbegin(), pose_sensor_->interface_names_.cend(),
41-
full_interface_names_.cbegin(), full_interface_names_.cend()));
39+
EXPECT_TRUE(
40+
std::equal(
41+
pose_sensor_->interface_names_.cbegin(), pose_sensor_->interface_names_.cend(),
42+
full_interface_names_.cbegin(), full_interface_names_.cend()));
4243

4344
// Get interface names
4445
std::vector<std::string> interface_names = pose_sensor_->get_state_interface_names();

‎controller_interface/test/test_semantic_component_command_interface.cpp

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,8 @@ TEST_F(SemanticCommandInterfaceTest, validate_command_interfaces)
7171
ASSERT_EQ(semantic_component_->command_interfaces_.size(), 0u);
7272

7373
// validate that release_interfaces() does not touch interface_names_
74-
ASSERT_TRUE(std::equal(
75-
semantic_component_->interface_names_.begin(), semantic_component_->interface_names_.end(),
76-
interface_names.begin(), interface_names.end()));
74+
ASSERT_TRUE(
75+
std::equal(
76+
semantic_component_->interface_names_.begin(), semantic_component_->interface_names_.end(),
77+
interface_names.begin(), interface_names.end()));
7778
}

‎controller_interface/test/test_semantic_component_interface.cpp

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -67,9 +67,10 @@ TEST_F(SemanticComponentInterfaceTest, validate_custom_names)
6767

6868
// validate the interface_names_
6969
std::vector<std::string> interface_names = semantic_component_->get_state_interface_names();
70-
ASSERT_TRUE(std::equal(
71-
semantic_component_->interface_names_.begin(), semantic_component_->interface_names_.end(),
72-
interface_names.begin(), interface_names.end()));
70+
ASSERT_TRUE(
71+
std::equal(
72+
semantic_component_->interface_names_.begin(), semantic_component_->interface_names_.end(),
73+
interface_names.begin(), interface_names.end()));
7374

7475
ASSERT_EQ(interface_names.size(), size_);
7576
ASSERT_EQ(interface_names[0], semantic_component_->test_name_ + "/i5");
@@ -129,7 +130,8 @@ TEST_F(SemanticComponentInterfaceTest, validate_state_interfaces)
129130
ASSERT_EQ(semantic_component_->state_interfaces_.size(), 0u);
130131

131132
// validate that release_interfaces() does not touch interface_names_
132-
ASSERT_TRUE(std::equal(
133-
semantic_component_->interface_names_.begin(), semantic_component_->interface_names_.end(),
134-
interface_names.begin(), interface_names.end()));
133+
ASSERT_TRUE(
134+
std::equal(
135+
semantic_component_->interface_names_.begin(), semantic_component_->interface_names_.end(),
136+
interface_names.begin(), interface_names.end()));
135137
}

‎controller_manager/src/controller_manager.cpp

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -388,8 +388,9 @@ ControllerManager::ControllerManager(
388388
: rclcpp::Node(manager_node_name, node_namespace, options),
389389
diagnostics_updater_(this),
390390
executor_(executor),
391-
loader_(std::make_shared<pluginlib::ClassLoader<controller_interface::ControllerInterface>>(
392-
kControllerInterfaceNamespace, kControllerInterfaceClassName)),
391+
loader_(
392+
std::make_shared<pluginlib::ClassLoader<controller_interface::ControllerInterface>>(
393+
kControllerInterfaceNamespace, kControllerInterfaceClassName)),
393394
chainable_loader_(
394395
std::make_shared<pluginlib::ClassLoader<controller_interface::ChainableControllerInterface>>(
395396
kControllerInterfaceNamespace, kChainableControllerInterfaceClassName)),
@@ -408,8 +409,9 @@ ControllerManager::ControllerManager(
408409
: rclcpp::Node(manager_node_name, node_namespace, options),
409410
diagnostics_updater_(this),
410411
executor_(executor),
411-
loader_(std::make_shared<pluginlib::ClassLoader<controller_interface::ControllerInterface>>(
412-
kControllerInterfaceNamespace, kControllerInterfaceClassName)),
412+
loader_(
413+
std::make_shared<pluginlib::ClassLoader<controller_interface::ControllerInterface>>(
414+
kControllerInterfaceNamespace, kControllerInterfaceClassName)),
413415
chainable_loader_(
414416
std::make_shared<pluginlib::ClassLoader<controller_interface::ChainableControllerInterface>>(
415417
kControllerInterfaceNamespace, kChainableControllerInterfaceClassName)),
@@ -430,8 +432,9 @@ ControllerManager::ControllerManager(
430432
resource_manager_(std::move(resource_manager)),
431433
diagnostics_updater_(this),
432434
executor_(executor),
433-
loader_(std::make_shared<pluginlib::ClassLoader<controller_interface::ControllerInterface>>(
434-
kControllerInterfaceNamespace, kControllerInterfaceClassName)),
435+
loader_(
436+
std::make_shared<pluginlib::ClassLoader<controller_interface::ControllerInterface>>(
437+
kControllerInterfaceNamespace, kControllerInterfaceClassName)),
435438
chainable_loader_(
436439
std::make_shared<pluginlib::ClassLoader<controller_interface::ChainableControllerInterface>>(
437440
kControllerInterfaceNamespace, kChainableControllerInterfaceClassName)),

0 commit comments

Comments
 (0)