Skip to content

Add minimal pre-commit config#406

Open
olantwin wants to merge 3 commits intoFramework-R-D:mainfrom
olantwin:pre-commit
Open

Add minimal pre-commit config#406
olantwin wants to merge 3 commits intoFramework-R-D:mainfrom
olantwin:pre-commit

Conversation

@olantwin
Copy link
Contributor

Implement #390 .

For now, only local hooks (no CI config).

For now ruff, clang-format and some of the hooks included with pre-commit.

I'll add cmake-format and jsonnet-fmt next.

@olantwin
Copy link
Contributor Author

@phlexbot yaml-fix

1 similar comment
@knoepfel
Copy link
Member

@phlexbot yaml-fix

@github-actions
Copy link
Contributor

Automatic YAML formatter fixes pushed (commit 6ac2d04).
⚠️ Note: Some issues may require manual review and fixing.

@knoepfel knoepfel linked an issue Mar 10, 2026 that may be closed by this pull request
@knoepfel
Copy link
Member

@olantwin, apologies about the YAML format problems. The changes in .github/workflows are not formally related to this PR.

@greenc-FNAL
Copy link
Contributor

Note: we use gersemi in place of cmake-format.

@olantwin
Copy link
Contributor Author

Note: we use gersemi in place of cmake-format.

Thanks for the pointer! Added.

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.

Using pre-commit hooks

3 participants