Merge pull request #18 from tier4/pre-commit-ci-update-config

ci(pre-commit): autoupdate

Author: drwnz

.pre-commit-config.yaml

@@ -4,7 +4,7 @@ ci:
 
 repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.1.0
+    rev: v4.5.0
     hooks:
       - id: check-json
       - id: check-merge-conflict
@@ -18,51 +18,51 @@ repos:
         args: [--markdown-linebreak-ext=md]
 
   - repo: https://github.com/igorshubovych/markdownlint-cli
-    rev: v0.30.0
+    rev: v0.37.0
     hooks:
       - id: markdownlint
         args: [-c, .markdownlint.yaml, --fix]
 
   - repo: https://github.com/pre-commit/mirrors-prettier
-    rev: v2.5.1
+    rev: v4.0.0-alpha.3
     hooks:
       - id: prettier
 
   - repo: https://github.com/adrienverge/yamllint
-    rev: v1.26.3
+    rev: v1.33.0
     hooks:
       - id: yamllint
 
   - repo: https://github.com/tier4/pre-commit-hooks-ros
-    rev: v0.4.0
+    rev: v0.8.0
     hooks:
       - id: prettier-package-xml
       - id: sort-package-xml
 
   - repo: https://github.com/shellcheck-py/shellcheck-py
-    rev: v0.8.0.3
+    rev: v0.9.0.6
     hooks:
       - id: shellcheck
 
   - repo: https://github.com/scop/pre-commit-shfmt
-    rev: v3.4.2-1
+    rev: v3.7.0-4
     hooks:
       - id: shfmt
         args: [-w, -s, -i=4]
 
   - repo: https://github.com/pycqa/isort
-    rev: 5.10.1
+    rev: 5.12.0
     hooks:
       - id: isort
 
   - repo: https://github.com/psf/black
-    rev: 22.1.0
+    rev: 23.11.0
     hooks:
       - id: black
         args: [--line-length=100]
 
   - repo: https://github.com/PyCQA/flake8
-    rev: 4.0.1
+    rev: 6.1.0
     hooks:
       - id: flake8
         additional_dependencies:
@@ -78,12 +78,12 @@ repos:
           ]
 
   - repo: https://github.com/pre-commit/mirrors-clang-format
-    rev: v13.0.0
+    rev: v17.0.6
     hooks:
       - id: clang-format
 
   - repo: https://github.com/cpplint/cpplint
-    rev: 1.5.5
+    rev: 1.6.1
     hooks:
       - id: cpplint
         args: [--quiet]

README.md

@@ -14,15 +14,15 @@ This repository provides a ROS2 package for generating sensor trigger signals on
 
 ## Installation
 
-1.  Create or change into your workspace directory, and execute the following:
+1. Create or change into your workspace directory, and execute the following:
 
     ```bash
     mkdir -p src
     git clone git@github.com:tier4/sensor_trigger.git src
     colcon build --symlink-install --cmake-args -DCMAKE_BUILD_TYPE=Release --packages-up-to sensor_trigger
     ```
 
-2.  The sensor trigger node requires as close to real-time operation as possible to maintain reliable trigger timing under heavy CPU load. If this is required, it is recommended to allow thread schedule priority setting to the ROS2 user by adding the following line to `/etc/security/limits.conf`:
+2. The sensor trigger node requires as close to real-time operation as possible to maintain reliable trigger timing under heavy CPU load. If this is required, it is recommended to allow thread schedule priority setting to the ROS2 user by adding the following line to `/etc/security/limits.conf`:
 
     ```
     <username>    -   rtprio   98

src/jetson_gpio.cpp

@@ -14,9 +14,10 @@
 
 #include <sensor_trigger/jetson_gpio.hpp>
 
-#include <cstdio>
 #include <sys/stat.h>
 
+#include <cstdio>
+
 namespace jetson_gpio
 {
 JetsonGpio::~JetsonGpio()
@@ -68,11 +69,11 @@ bool JetsonGpio::export_gpio()
 
   // Check target GPIO is available (not opened) first.
   // If it is unavailable, close the port and try to open it.
-    std::string target_gpio = std::string(SYSFS_GPIO_DIR) + std::string("/gpio")
-                              + std::to_string(gpio_);
+  std::string target_gpio =
+    std::string(SYSFS_GPIO_DIR) + std::string("/gpio") + std::to_string(gpio_);
   struct stat return_status;
   if (stat(target_gpio.c_str(), &return_status) == 0) {
-       unexport_gpio();
+    unexport_gpio();
   }
 
   file_descriptor = open(SYSFS_GPIO_DIR "/export", O_WRONLY);