Skip to content
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

Use darker as code formatter tool base... #74

Merged
merged 2 commits into from
Feb 1, 2022
Merged

Use darker as code formatter tool base... #74

merged 2 commits into from
Feb 1, 2022

Conversation

jedie
Copy link
Collaborator

@jedie jedie commented Feb 1, 2022

Use Darker as code formatter. Darker is:
"""
Apply black reformatting to Python files only in regions changed since a given commit.
"""

Also use pytest-darker, pytest-flake8 and pytest-isort ... So code formatting "errors" will be
check in pytest run. (Note: Only the first test is slower, afer one test runs only changed files
will be checked.)

Based on #73

Jens Diemer added 2 commits February 1, 2022 09:56
Use Darker as code formatter. Darker is:
"""
Apply black reformatting to Python files only in regions changed since a given commit.
"""

Also use `pytest-darker`, `pytest-flake8` and `pytest-isort` ... So code formatting "errors" will be
check in pytest run. (Note: Only the first test is slower, afer one test runs only changed files
will be checked.)
@codecov-commenter
Copy link

codecov-commenter commented Feb 1, 2022

Codecov Report

Merging #74 (2e81ed1) into master (e147acd) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #74   +/-   ##
=======================================
  Coverage   83.56%   83.56%           
=======================================
  Files          35       35           
  Lines         791      791           
=======================================
  Hits          661      661           
  Misses        130      130           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e147acd...2e81ed1. Read the comment docs.

@phihag phihag merged commit 01d94fa into master Feb 1, 2022
@jedie jedie deleted the darker branch February 1, 2022 09:28
jedie pushed a commit that referenced this pull request Feb 3, 2022
Code check is done in pytest run, since: #74

And there is a problem with 'fix-code-style', because of:
akaihola/darker#273 (black would like to access docker volumes and raise a
permission error)
jedie pushed a commit that referenced this pull request Feb 3, 2022
There is no need to test linting separately, because this is done in pytest run since:
#74
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.

3 participants