Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add clang-format 19 #82

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add clang-format 19 #82

wants to merge 1 commit into from

Conversation

d-griet
Copy link

@d-griet d-griet commented Dec 20, 2024

This PR adds support for clang-format 19.

Signed-off-by: Darrel Griët <d.griet@dynatron.nl>
@N-Dekker
Copy link

@DoozyX @d-griet Great work! Are there any plans to also add clang-format version 19.1? Preferably 19.1.4, as that's the version required by ITK (pull request InsightSoftwareConsortium/ITK#5015).

N-Dekker added a commit to SuperElastix/elastix that referenced this pull request Feb 11, 2025
Dropped https://github.com/DoozyX/clang-format-lint-action in order to upgrade to clang-format 19.1.4, following ITK pull request InsightSoftwareConsortium/ITK#5015

Reverts commit 8ac0413, "STYLE: add clang format check", by Viktor van der Valk, 11 Dec 2020.

Note: there is a pull request to add clang-format 19 to DoozyX/clang-format-lint-action, by Darrel Griët: DoozyX/clang-format-lint-action#82
N-Dekker added a commit to SuperElastix/elastix that referenced this pull request Feb 11, 2025
Dropped https://github.com/DoozyX/clang-format-lint-action in order to upgrade to clang-format 19.1.4, following ITK pull request InsightSoftwareConsortium/ITK#5015

Reverts commit 8ac0413, "STYLE: add clang format check", by Viktor van der Valk, 11 Dec 2020.

Note: there is a pull request to add clang-format 19 to DoozyX/clang-format-lint-action, by Darrel Griët: DoozyX/clang-format-lint-action#82
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants