Skip to content

Use prek in documentation and CI#22505

Merged
charliermarsh merged 1 commit intomainfrom
charlie/prek
Jan 11, 2026
Merged

Use prek in documentation and CI#22505
charliermarsh merged 1 commit intomainfrom
charlie/prek

Conversation

@charliermarsh
Copy link
Member

Summary

AFAIK, many of us on the team are using prek now. It seems appropriate to modify the docs to formalize it.

@charliermarsh charliermarsh added the documentation Improvements or additions to documentation label Jan 11, 2026
@AlexWaygood AlexWaygood changed the title Use prek in docuemntation Use prek in documentation Jan 11, 2026
cargo fetch

pip install maturin pre-commit
pip install maturin prek
Copy link
Member

Choose a reason for hiding this comment

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

what's pip

Copy link
Member Author

Choose a reason for hiding this comment

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

Lol

Copy link
Member Author

Choose a reason for hiding this comment

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

(I don't dare touch .devcontainer beyond this.)

@charliermarsh charliermarsh marked this pull request as ready for review January 11, 2026 18:35
Copy link
Member

@AlexWaygood AlexWaygood left a comment

Choose a reason for hiding this comment

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

You could also update these comments:

# To run all hooks *including* this hook, use `uvx pre-commit run -a --hook-stage=manual`.
# To run *just* this hook, use `uvx pre-commit run -a actionlint --hook-stage=manual`.

@charliermarsh charliermarsh force-pushed the charlie/prek branch 2 times, most recently from 8b8d147 to 2eb673a Compare January 11, 2026 18:47
@AlexWaygood AlexWaygood changed the title Use prek in documentation Use prek in documentation and CI Jan 11, 2026
@astral-sh-bot
Copy link

astral-sh-bot bot commented Jan 11, 2026

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

Formatter (stable)

✅ ecosystem check detected no format changes.

Formatter (preview)

✅ ecosystem check detected no format changes.

@charliermarsh charliermarsh merged commit 7bacca9 into main Jan 11, 2026
45 checks passed
@charliermarsh charliermarsh deleted the charlie/prek branch January 11, 2026 19:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments