Skip to content

Conversation

@vkucera
Copy link
Contributor

@vkucera vkucera commented Jan 24, 2025

No description provided.

@vkucera vkucera marked this pull request as ready for review January 24, 2025 08:25
@vkucera vkucera changed the title Configure formatting tools. Add pre-commit hooks. Configure linters and formattiers. Add pre-commit hooks. Jan 24, 2025
@vkucera vkucera changed the title Configure linters and formattiers. Add pre-commit hooks. Configure linters and formatters. Add pre-commit hooks. Jan 24, 2025
@vkucera vkucera changed the title Configure linters and formatters. Add pre-commit hooks. Configure linters and formatters as pre-commit hooks and in MegaLinter Jan 24, 2025
@vkucera vkucera changed the title Configure linters and formatters as pre-commit hooks and in MegaLinter Configure linters and formatters in MegaLinter and as pre-commit hooks Jan 24, 2025
Copy link
Member

@qgp qgp left a comment

Choose a reason for hiding this comment

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

Thanks @vkucera ! Testing this now locally.

@qgp qgp merged commit 0e7a4e3 into alisw:run3 Feb 7, 2025
1 check passed
@vkucera vkucera deleted the pre-commit branch February 9, 2025 22:40
vkucera added a commit to vkucera/MachineLearningHEP that referenced this pull request Feb 12, 2025
alisw#970)

* Add pre-commit

* Add default hooks

* Add more hooks

* Format

* Update

* Fix ruff

* Sort imports with ruff

* Add yaml hooks

* Configure yaml format

* Do not fail yamllint for indentation

* Add yaml linters back and remove redundant python linters

* Adjust yaml style

* Disable prettier

* Adjust comments

* Add line

* Rename yamllint config file

* Rename yamlfmt config file

* Remove pre-commit from requirements.txt

---------

Co-authored-by: Jochen Klein <[email protected]>
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