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

Black update #945

Merged
merged 1 commit into from
Apr 1, 2022
Merged

Black update #945

merged 1 commit into from
Apr 1, 2022

Conversation

TheChymera
Copy link
Contributor

Older black will hit an error if the system uses click 8.1.0 or newer.
See: psf/black#2964 (comment)

@yarikoptic this is pertaining to the issue we discussed earlier this evening, and this commit solves it.

Older black will hit an error if the system uses click 8.1.0 or newer.
See: psf/black#2964 (comment)
@codecov
Copy link

codecov bot commented Apr 1, 2022

Codecov Report

Merging #945 (293a0f9) into master (d2868f5) will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #945      +/-   ##
==========================================
+ Coverage   87.24%   87.27%   +0.02%     
==========================================
  Files          62       62              
  Lines        7709     7709              
==========================================
+ Hits         6726     6728       +2     
+ Misses        983      981       -2     
Flag Coverage Δ
unittests 87.27% <ø> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
dandi/support/threaded_walk.py 92.85% <0.00%> (-1.79%) ⬇️
dandi/organize.py 81.85% <0.00%> (-0.45%) ⬇️
dandi/_version.py 45.96% <0.00%> (+1.75%) ⬆️

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 d2868f5...293a0f9. Read the comment docs.

@jwodder jwodder added the internal Changes only affect the internal API label Apr 1, 2022
@jwodder jwodder merged commit c8de551 into master Apr 1, 2022
@jwodder jwodder deleted the black branch April 1, 2022 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
internal Changes only affect the internal API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants