Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

pre commit action #1293

Merged
merged 6 commits into from
Feb 13, 2025
Merged

pre commit action #1293

merged 6 commits into from
Feb 13, 2025

Conversation

thewtex
Copy link
Contributor

@thewtex thewtex commented Feb 11, 2025

  • ENH: Add pre-commit GitHub Action
  • ENH: Bump clang-format to v19.1.7
  • STYLE: pre-commit end-of-file and trailing whitespace fixes
  • ENH: Bump pre-commit to 4.1.0

@thewtex
ENH: Add pre-commit GitHub Action
87446b7 
Run clang-format and other checks on pull requests.
@thewtex
ENH: Bump clang-format to v19.1.7
d01c08d 
Lastest version in mirrors-clang-format and version updated for ITK.
@thewtex
STYLE: pre-commit end-of-file and trailing whitespace fixes
f8030e6 
Applied via:

  ./.pixi/envs/pre-commit/bin/pre-commit  run --all-files
@thewtex
ENH: Bump pre-commit to 4.1.0
6ed693f 
Latest version.
@thewtex
Copy link
Contributor Author

thewtex commented Feb 11, 2025 •
edited
Loading

Closes #1292

@thewtex thewtex requested a review from N-Dekker February 11, 2025 17:53
@thewtex
STYLE: Remove forbit-submodules pre-commit check
a90fd1e 
We have an existing dox/manual submodule.
@thewtex
STYLE: Trailing-whitespace fixes from pre-commit
5ae8a05 
After 4.1.0 pre-commit update,

  ./.pixi/envs/pre-commit/bin/pre-commit  run --all-files
N-Dekker
N-Dekker approved these changes Feb 13, 2025
View reviewed changes
Copy link
Member

@N-Dekker N-Dekker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you very much, Matt!

@N-Dekker
Copy link
Member

@N-Dekker N-Dekker merged commit 17eb1ca into SuperElastix:main Feb 13, 2025
5 checks passed
@N-Dekker N-Dekker mentioned this pull request Feb 13, 2025
Merged
@thewtex thewtex deleted the pre-commit-action branch February 13, 2025 16:12
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@N-Dekker N-Dekker N-Dekker approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@thewtex @N-Dekker