Skip to content

Commit

Permalink
Update clang standard
Browse files Browse the repository at this point in the history
clang 15 have a lot of fix and improvements :
https://releases.llvm.org/15.0.0/tools/clang/docs/ReleaseNotes.html

this patch also update
psf/black and pre-commit-hooks
  • Loading branch information
mosfet80 committed Nov 3, 2024
1 parent d5f763e commit 6bcd0bc
Show file tree
Hide file tree
Showing 6 changed files with 13 additions and 13 deletions.
2 changes: 1 addition & 1 deletion .docker/ci-testing/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ RUN \
# Some basic requirements
wget git sudo curl \
# Preferred build tools
clang clang-format-14 clang-tidy clang-tools \
clang clang-format-15 clang-tidy clang-tools \
ccache \
ros-"$ROS_DISTRO"-ros-base && \
#
Expand Down
2 changes: 1 addition & 1 deletion .docker/ci/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ RUN \
# Some basic requirements
wget git sudo curl \
# Preferred build tools
clang clang-format-14 clang-tidy clang-tools \
clang clang-format-15 clang-tidy clang-tools \
ccache && \
#
# Globally disable git security
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/format.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,8 @@ jobs:
- uses: actions/setup-python@v5
with:
python-version: 3.x
- name: Install clang-format-14
run: sudo apt-get install clang-format-14
- name: Install clang-format-15
run: sudo apt-get install clang-format-15
- uses: pre-commit/action@v3.0.1
id: precommit
- name: Upload pre-commit changes
Expand Down
6 changes: 3 additions & 3 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
repos:
# Standard hooks
- repo: https://github.com/pre-commit/pre-commit-hooks
rev: v4.5.0
rev: v5.0.0
hooks:
- id: check-added-large-files
- id: check-case-conflict
Expand All @@ -33,7 +33,7 @@ repos:
- id: trailing-whitespace

- repo: https://github.com/psf/black
rev: 23.10.0
rev: 24.10.0
hooks:
- id: black

Expand All @@ -42,7 +42,7 @@ repos:
- id: clang-format
name: clang-format
description: Format files with ClangFormat.
entry: clang-format-14
entry: clang-format-15
language: system
files: \.(c|cc|cxx|cpp|frag|glsl|h|hpp|hxx|ih|ispc|ipp|java|js|m|proto|vert)$
args: ['-fallback-style=none', '-i']
Expand Down
6 changes: 3 additions & 3 deletions moveit/scripts/create_readme_table.py
Original file line number Diff line number Diff line change
Expand Up @@ -62,9 +62,9 @@ def define_urls(target, params):
)
params["url"] = "{base_url}/view/{R}src_u{U}/job/{job}".format(**params)
elif target == "bin":
params[
"job"
] = "{R}bin_u{U}64__{package}__ubuntu_{ubuntu}_amd64__binary".format(**params)
params["job"] = (
"{R}bin_u{U}64__{package}__ubuntu_{ubuntu}_amd64__binary".format(**params)
)
params["url"] = "{base_url}/view/{R}bin_u{U}64/job/{job}".format(**params)


Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -67,8 +67,8 @@ def perform(self, context: LaunchContext) -> Text:
for key, value in self.__mappings.items():
normalized_key = normalize_to_list_of_substitutions(key)
normalized_value = normalize_to_list_of_substitutions(value)
expanded_mappings[
perform_substitutions(context, normalized_key)
] = perform_substitutions(context, normalized_value)
expanded_mappings[perform_substitutions(context, normalized_key)] = (
perform_substitutions(context, normalized_value)
)

return load_xacro(Path(expanded_file_path), mappings=expanded_mappings)

0 comments on commit 6bcd0bc

Please sign in to comment.