Skip to content

Enable pre-commit checks in CI and fix findings#125

Merged
AlexanderLanin merged 15 commits intoeclipse-score:mainfrom
elektrobit-contrib:fix-pre-commit-hooks-all-findings
Mar 9, 2026
Merged

Enable pre-commit checks in CI and fix findings#125
AlexanderLanin merged 15 commits intoeclipse-score:mainfrom
elektrobit-contrib:fix-pre-commit-hooks-all-findings

Conversation

@lurtz
Copy link
Contributor

@lurtz lurtz commented Mar 8, 2026

With #117 this repo got a pre-commit configuration, but it was not enforced, because there were many violations and to keep the scope of the pull request somewhat smaller. Now is the same for fixes.

Most changes are copyright headers. cr_checker is not yet perfect and I used reuse annotate for the files, which cr_checker does not recognize.

@lurtz lurtz changed the title Run all pre-commit checks in CI Enable pre-commit checks in CI and fix findings Mar 8, 2026
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this file is not yet recognized by cr_checker and was fixed with pipx run reuse annotate

@lurtz lurtz marked this pull request as ready for review March 8, 2026 18:57
@AlexanderLanin AlexanderLanin merged commit 2f5c7a8 into eclipse-score:main Mar 9, 2026
5 checks passed
@lurtz lurtz deleted the fix-pre-commit-hooks-all-findings branch March 9, 2026 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants