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

Reformat using black formatting #10

Merged
merged 5 commits into from
Aug 15, 2022
Merged

Reformat using black formatting #10

merged 5 commits into from
Aug 15, 2022

Conversation

jlaehne
Copy link
Contributor

@jlaehne jlaehne commented Aug 14, 2022

Description of the change

With all relevant PRs having been dealt with in the course of the split, I think it is a good occasion to enforce black formatting for the repo.

Used the black command to reformat all .py files.

Progress of the PR

  • Change implemented (can be split into several points),
  • add black lint test to PR-actions
  • [na] add an changelog entry in the upcoming_changes folder (see upcoming_changes/README.rst),
  • [na] Check formatting changelog entry in the readthedocs doc build of this PR (link in github checks)
  • ready for review.

@codecov
Copy link

codecov bot commented Aug 14, 2022

Codecov Report

Merging #10 (892bf4e) into main (77e79b0) will not change coverage.
The diff coverage is 76.14%.

@@           Coverage Diff           @@
##             main      #10   +/-   ##
=======================================
  Coverage   76.54%   76.54%           
=======================================
  Files          39       39           
  Lines        8336     8336           
  Branches     1931     1931           
=======================================
  Hits         6381     6381           
  Misses       1500     1500           
  Partials      455      455           
Impacted Files Coverage Δ
rsciio/mrc/api.py 0.00% <0.00%> (ø)
rsciio/netcdf/api.py 0.00% <0.00%> (ø)
rsciio/tiff/api.py 88.31% <ø> (ø)
rsciio/tvips/api.py 98.46% <ø> (ø)
rsciio/usid_hdf5/api.py 0.00% <ø> (ø)
rsciio/utils/date_time_tools.py 95.74% <ø> (ø)
rsciio/utils/fei_stream_readers.py 90.90% <ø> (ø)
rsciio/utils/rgb_tools.py 79.62% <ø> (ø)
rsciio/utils/skimage_exposure.py 62.50% <ø> (ø)
rsciio/utils/tools.py 71.42% <ø> (ø)
... and 37 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@jlaehne jlaehne requested a review from ericpre August 14, 2022 20:42
# Conflicts:
#	rsciio/exceptions.py
#	rsciio/sur/api.py
@ericpre ericpre merged commit 4ff3f4a into hyperspy:main Aug 15, 2022
@jlaehne jlaehne deleted the black branch August 15, 2022 15:43
@jlaehne jlaehne added this to the v0.1.0 initial release milestone Aug 29, 2022
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