Skip to content

Use new ruff-check pre-commit ID, update version#66

Merged
ax3l merged 1 commit intopals-project:mainfrom
EZoni:ruff_precommit_id
Apr 1, 2026
Merged

Use new ruff-check pre-commit ID, update version#66
ax3l merged 1 commit intopals-project:mainfrom
EZoni:ruff_precommit_id

Conversation

@EZoni
Copy link
Copy Markdown
Member

@EZoni EZoni commented Mar 30, 2026

Just found out that the new pre-commit ID ruff-check was introduced a while ago in astral-sh/ruff-pre-commit#124 and seems to be preferred to the old ID ruff. See also ruff-pre-commit's main README file github.com/astral-sh/ruff-pre-commit/blob/main/README.md.

@EZoni EZoni added the CI/CD label Mar 30, 2026
- repo: https://github.com/astral-sh/ruff-pre-commit
# Ruff version
rev: v0.9.6
rev: v0.15.7
Copy link
Copy Markdown
Member Author

@EZoni EZoni Mar 30, 2026

Choose a reason for hiding this comment

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

@ax3l

Note that I had to update the version manually, because it does not get auto-updated (as in, e.g., BLAST-WarpX/warpx#6700) even though pre-commit.ci is set up correctly. It may have something to do with some settings of the original repository from which pals-python was forked.

@EZoni EZoni requested a review from ax3l March 30, 2026 17:28
Copy link
Copy Markdown
Member

@ax3l ax3l 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! :)

@ax3l ax3l merged commit 5ff1f2d into pals-project:main Apr 1, 2026
16 checks passed
@EZoni EZoni deleted the ruff_precommit_id branch April 1, 2026 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants