ci: check include guard

[isamu-takagi]

Related Issue(required)

None

Description(required)

Add pre-commit for include-guard.

Review Procedure(required)

pre-commit run -a

Related PR(optional)

tier4/pre-commit-hooks-ros#32

Pre-Review Checklist for the PR Author

PR Author should check the checkboxes below when creating the PR.

• Read commit-guidelines
• Assign PR to reviewer

If you are adding new package following items are required:

• Documentation with description of the package is available
• A sample launch file and parameter file are available if the package contains executable nodes

Checklist for the PR Reviewer

Reviewers should check the checkboxes below before approval.

• Commits are properly organized and messages are according to the guideline
• PR title describes the changes

Post-Review Checklist for the PR Author

PR Author should check the checkboxes below before merging.

• All open points are addressed and tracked via issues or tickets

CI Checks

• Build and test for PR / build-and-test-pr: Required to pass before the merge.
• Build and test for PR / clang-tidy-pr: NOT required to pass before the merge. It is up to the reviewer(s). Found false positives? See the [guidelines][clang-tidy-guidelines].
• Check spelling: NOT required to pass before the merge. It is up to the reviewer(s). See here if you want to add some words to the spell check dictionary.

[kenji-miyake]

🤔

~/ghq/github.com/autowarefoundation/autoware.universe feature/pre-commit-include-guard
❯ pre-commit run -a
[WARNING] The 'rev' field of repo 'https://github.com/tier4/pre-commit-hooks-ros' appears to be a mutable reference (moving tag / branch).  Mutable references are never updated after first install and are not su
pported.  See https://pre-commit.com/#using-the-latest-version-for-a-repository for more details.  Hint: `pre-commit autoupdate` often fixes this.
[INFO] Initializing environment for https://github.com/tier4/pre-commit-hooks-ros.
[INFO] Initializing environment for https://github.com/tier4/pre-commit-hooks-ros:@prettier/plugin-xml@1.1.0,prettier@2.4.1.
[INFO] Installing environment for https://github.com/tier4/pre-commit-hooks-ros.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
An unexpected error has occurred: AssertionError:
Check the log at /home/kenji/.cache/pre-commit/pre-commit.log

The log is:

### version information

```
pre-commit version: 2.15.0
sys.version:
    3.8.10 (default, Nov 26 2021, 20:14:08) 
    [GCC 9.3.0]
sys.executable: /usr/bin/python3
os.name: posix
sys.platform: linux
```

### error information

```
An unexpected error has occurred: AssertionError: 
```

```
Traceback (most recent call last):
  File "/home/kenji/.local/lib/python3.8/site-packages/pre_commit/error_handler.py", line 65, in error_handler
    yield
  File "/home/kenji/.local/lib/python3.8/site-packages/pre_commit/main.py", line 396, in main
    return run(args.config, store, args)
  File "/home/kenji/.local/lib/python3.8/site-packages/pre_commit/commands/run.py", line 417, in run
    install_hook_envs(to_install, store)
  File "/home/kenji/.local/lib/python3.8/site-packages/pre_commit/repository.py", line 224, in install_hook_envs
    _hook_install(hook)
  File "/home/kenji/.local/lib/python3.8/site-packages/pre_commit/repository.py", line 82, in _hook_install
    lang.install_environment(
  File "/home/kenji/.local/lib/python3.8/site-packages/pre_commit/languages/node.py", line 85, in install_environment
    assert prefix.exists('package.json')
AssertionError
```

-> This was because of tier4/pre-commit-hooks-ros#37.

[isamu-takagi]

This error occurs when using branch name or commit hash for rev.
Do we need to create a tag on tier4/pre-commit-hooks-ros?

This was because there was no package.json.

[isamu-takagi]

I confirmed it with the try-repo command.
pre-commit try-repo https://github.com/tier4/pre-commit-hooks-ros ros-include-guard -a --ref ros-include-guard

[kenji-miyake]

Oh, I didn't know try-repo command! It's nice!

Do we need to create a tag on tier4/pre-commit-hooks-ros?

Not necessary for now. Let's create a new tag after your PR is merged.

[kenji-miyake]

The rest looks fine.

[codecov]

Codecov Report

Merging #438 (a1294c0) into tier4/proposal (5f058a8) will decrease coverage by 0.74%.
The diff coverage is n/a.

@@                Coverage Diff                 @@
##           tier4/proposal     #438      +/-   ##
==================================================
- Coverage           10.99%   10.25%   -0.75%     
==================================================
  Files                 705      514     -191     
  Lines               49462    38569   -10893     
  Branches             6658     5233    -1425     
==================================================
- Hits                 5438     3954    -1484     
+ Misses              39663    31234    -8429     
+ Partials             4361     3381     -980     

Impacted Files	Coverage Δ
...point_types/include/autoware_point_types/types.hpp	30.00% <ø> (ø)
...tier4_autoware_utils/trajectory/tmp_conversion.hpp	66.66% <ø> (ø)
localization/ndt/include/ndt/impl/base.hpp	0.00% <ø> (ø)
localization/ndt/include/ndt/impl/omp.hpp	0.00% <ø> (ø)
localization/ndt/include/ndt/impl/pcl_generic.hpp	0.00% <ø> (ø)
localization/ndt/include/ndt/impl/pcl_modified.hpp	0.00% <ø> (ø)
localization/ndt/include/ndt/omp.hpp	0.00% <ø> (ø)
localization/ndt/include/ndt/pcl_generic.hpp	0.00% <ø> (ø)
localization/ndt/include/ndt/pcl_modified.hpp	0.00% <ø> (ø)
...pcl_modified/include/ndt_pcl_modified/impl/ndt.hpp	0.00% <ø> (ø)
... and 203 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5f058a8...a1294c0. Read the comment docs.