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 more official pre-commit hooks #255

Merged
merged 2 commits into from
Jan 28, 2025

Conversation

jbampton
Copy link
Member

@Pilot-Pirx
Copy link
Member

I would also be interested in:
https://github.com/pre-commit/pre-commit-hooks?tab=readme-ov-file#mixed-line-ending

I already found and fixed some files with mixed line endings, but I could imagine that there are more of them...

@jbampton
Copy link
Member Author

I pushed a second commit that fixed the line mixed line ending in main for just cxx and hxx files.

Was a large amount just for this batch

@jbampton jbampton changed the title Add 4 more official pre-commit hooks Add more official pre-commit hooks Jan 24, 2025
@Pilot-Pirx Pilot-Pirx merged commit 36d8fc7 into apache:trunk Jan 28, 2025
1 check passed
asfgit pushed a commit that referenced this pull request Jan 28, 2025
* Add 4 more official pre-commit hooks

https://github.com/pre-commit/pre-commit-hooks

* Fix mixed line endings in `cxx` and `hxx` in `main`

(cherry picked from commit 36d8fc7)
@jbampton jbampton deleted the pre-commit-add-more-hooks branch January 28, 2025 14:17
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