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

Pin black version #251

Merged
merged 1 commit into from
Aug 29, 2020
Merged

Pin black version #251

merged 1 commit into from
Aug 29, 2020

Conversation

kylebarron
Copy link
Member

@kylebarron kylebarron commented Aug 29, 2020

CI on master is currently failing with

black....................................................................Failed
- hook id: black
- exit code: 123
- files were modified by this hook

reformatted /home/runner/work/rio-tiler/rio-tiler/rio_tiler/expression.py
reformatted /home/runner/work/rio-tiler/rio-tiler/docs/scripts/colormap_thumb.py
reformatted /home/runner/work/rio-tiler/rio-tiler/rio_tiler/io/base.py
error: cannot format /home/runner/work/rio-tiler/rio-tiler/rio_tiler/io/cogeo.py: INTERNAL ERROR: Black produced code that is not equivalent to the source.  Please report a bug on https://github.com/psf/black/issues.  This diff might be helpful: /tmp/blk_lzqera7f.log
error: cannot format /home/runner/work/rio-tiler/rio-tiler/rio_tiler/io/stac.py: INTERNAL ERROR: Black produced code that is not equivalent to the source.  Please report a bug on https://github.com/psf/black/issues.  This diff might be helpful: /tmp/blk_5b3wznne.log
reformatted /home/runner/work/rio-tiler/rio-tiler/rio_tiler/tasks.py
reformatted /home/runner/work/rio-tiler/rio-tiler/rio_tiler/reader.py
reformatted /home/runner/work/rio-tiler/rio-tiler/tests/test_io_cogeo.py
Oh no! 💥 💔 💥
6 files reformatted, 27 files left unchanged, 2 files failed to reformat.

This may be related to psf/black#1629.

In any case, good to pin the black version.

@vincentsarago vincentsarago merged commit e26cda7 into master Aug 29, 2020
@kylebarron kylebarron deleted the kyle/pin-black branch August 29, 2020 15:22
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