-
Notifications
You must be signed in to change notification settings - Fork 48
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Create initial pre-commit
configuration
#165
base: main
Are you sure you want to change the base?
Conversation
The commit introduces a basic `pre-commit` configuration with hooks for auto-fixing common code quality issues: * Ensure files end with a newline (`end-of-file-fixer`), * Standardizing line endings to LF (`mixed-line-ending`), and * Removing trailing whitespace (`trailing-whitespace`). To activate, developers should install `pre-commit` and `pre-commit install`. By using the https://pre-commit.ci application, the repository can ensure compliance friction-free. Signed-off-by: Stavros Ntentos <133706+stdedos@users.noreply.github.com>
Here is my proposal. I hope upstream (i.e., you, in plural) is okay with installing the https://pre-commit.ci/ app. Please familiarize yourself with the (short) website/demo/docs https://pre-commit.ci/. It may be possible to see the results of my MR, after you install the pre-commit.ci in the repository, on this same MR. |
There are doubts in this regard:
If you agree to include an email address in your GitHub profile, I can invite you on Slack to discuss with the repository owner. |
@isaak654 I have tried reaching you in private on Discord |
README.md
Outdated
Specifically, for this repo: | ||
* Install `pre-commit` (`pip install pre-commit`; see more ways in https://pre-commit.com/#installation), and | ||
* At the root of the repository, run `pre-commit install` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Specifically, for this repo: | |
* Install `pre-commit` (`pip install pre-commit`; see more ways in https://pre-commit.com/#installation), and | |
* At the root of the repository, run `pre-commit install` | |
Specifically, for this repository: | |
* Install `pre-commit` (`pip install pre-commit`; see more ways in https://pre-commit.com/#installation) | |
* At the root of the repository, run `pre-commit install` |
Signed-off-by: Stavros Ntentos <133706+stdedos@users.noreply.github.com>
The commit introduces a basic
pre-commit
configurationwith hooks for auto-fixing common code quality issues:
end-of-file-fixer
),mixed-line-ending
), andtrailing-whitespace
).To activate, developers should install
pre-commit
andpre-commit install
.By using the https://pre-commit.ci application, the repository can ensure compliance friction-free.
Signed-off-by: Stavros Ntentos 133706+stdedos@users.noreply.github.com