Skip to content

ci: configure pre-commit.ci to automate PR formatting (#894) - #1918

Open
prateek-dagar wants to merge 1 commit into
py-pdf:masterfrom
prateek-dagar:feature/enable-precommit-ci
Open

ci: configure pre-commit.ci to automate PR formatting (#894)#1918
prateek-dagar wants to merge 1 commit into
py-pdf:masterfrom
prateek-dagar:feature/enable-precommit-ci

Conversation

@prateek-dagar

Copy link
Copy Markdown

Addresses #894

Adds the pre-commit.ci configuration block to .pre-commit-config.yaml to automate formatting checks and auto-fixes for future Pull Requests.

  • Automates PR formatting auto-fixes using Black.
  • Bypasses the local system pylint hook (as system hooks are not supported in the sandboxed CI environment).

Checklist:

  • A unit test is covering the code added / modified by this PR N/A
  • In case of a new feature, docstrings have been added, with also some documentation in the docs/ folder N/A
  • A mention of the change is present in CHANGELOG.md N/A
  • This PR is ready to be merged

@andersonhc

Copy link
Copy Markdown
Collaborator

Thank you @prateek-dagar

Please allow me some time to setup this on a fork and submit a few test commits to test how it works, I'll get back to you as soon as I have some results.

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