Skip to content

Pre-commit configuration for black formatter#387

Open
krestenlaust wants to merge 5 commits into
nextfrom
setup-precommit
Open

Pre-commit configuration for black formatter#387
krestenlaust wants to merge 5 commits into
nextfrom
setup-precommit

Conversation

@krestenlaust
Copy link
Copy Markdown
Member

@krestenlaust krestenlaust commented Nov 4, 2023

Having pre-commit set-up would allow contributors to easily configure black formatter to run before every commit.

Though I haven't been able to get it working on my (windows) machine.

@krestenlaust krestenlaust changed the title Pre-commit hook configuration for black formatter Pre-commit configuration for black formatter Nov 4, 2023
@codecov
Copy link
Copy Markdown

codecov Bot commented Nov 4, 2023

Codecov Report

Merging #387 (0431489) into next (4c9a790) will not change coverage.
The diff coverage is n/a.

❗ Current head 0431489 differs from pull request most recent head fa13dab. Consider uploading reports for the commit fa13dab to get more accurate results

@@           Coverage Diff           @@
##             next     #387   +/-   ##
=======================================
  Coverage   53.49%   53.49%           
=======================================
  Files          33       33           
  Lines        4544     4544           
  Branches      267      267           
=======================================
  Hits         2431     2431           
  Misses       2063     2063           
  Partials       50       50           

@ThomasBow
Copy link
Copy Markdown
Contributor

@krestenlaust review changes

@ThomasBow ThomasBow marked this pull request as ready for review May 19, 2026 22:46
Copy link
Copy Markdown
Member Author

@krestenlaust krestenlaust left a comment

Choose a reason for hiding this comment

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

@ThomasBow will you be able to add something short to the readme about installing the pre-commit hook?

@ThomasBow
Copy link
Copy Markdown
Contributor

@ThomasBow will you be able to add something short to the readme about installing the pre-commit hook?

That is probably a good idea

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants