Skip to content

Conversation

DhavalGojiya
Copy link
Member

Added a Ruff formatting step to the CI pipeline.

@DhavalGojiya DhavalGojiya requested review from cclauss and cunla October 13, 2025 09:09
@cclauss
Copy link
Contributor

cclauss commented Oct 13, 2025

Perhaps we should consider running https://pre-commit.com and https://pre-commit.ci so that fast linting steps are done both locally on developers' machines and on GitHub CI. A pre-commit file might look something like:
https://raw.githubusercontent.com/ISKME/Open-Metadata-Exchange/refs/heads/main/.pre-commit-config.yaml

Thoughts?

@DhavalGojiya
Copy link
Member Author

DhavalGojiya commented Oct 13, 2025

Perhaps we should consider running https://pre-commit.com and https://pre-commit.ci so that fast linting steps are done both locally on developers' machines and on GitHub CI. A pre-commit file might look something like: https://raw.githubusercontent.com/ISKME/Open-Metadata-Exchange/refs/heads/main/.pre-commit-config.yaml

Thoughts?

Also thinking the same - it will be a good step to catch the checks early.

How about using Prek? It's written in Rust, much faster, and an alternative to pre-commit (Python).

@cclauss cclauss merged commit 1abae00 into master Oct 13, 2025
20 checks passed
@cclauss cclauss deleted the ci/add-ruff-formatter-step branch October 13, 2025 11:43
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