Skip to content

Should this repo use pre-commit ot run linters, formatters, typo checkers, etc? #315

@cclauss

Description

@cclauss

Is your feature request related to a problem? Please describe.
I'm always frustrated when I push code changes, but the CI fails because of linter, formatter, or typo checks

Describe the solution you'd like
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

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Prek? It's written in Rust, much faster, and an alternative to pre-commit (Python).

This might be a bit too bleeding-edge compared to the broadly used and stable pre-commit, but I expect Prek will make more sense in the near future.

Additional context
Add any other context or screenshots about the feature request here.

@DhavalGojiya

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions