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

Fix precommit issue between black and flake #101

Closed
mmeendez8 opened this issue Jan 27, 2021 · 2 comments
Closed

Fix precommit issue between black and flake #101

mmeendez8 opened this issue Jan 27, 2021 · 2 comments
Assignees
Labels
bug Something isn't working CI/CD Continuous Integration / Continuous Deployment

Comments

@mmeendez8
Copy link
Contributor

mmeendez8 commented Jan 27, 2021

Code to reproduce:

    image_0[
        int(image_0_bbox_top) : int(image_0_bbox_top + image_0_bbox_height),
        int(image_0_bbox_left) : int(image_0_bbox_left + image_0_bbox_width),
    ] = 0

Output:

flake8...................................................................Failed
- hook id: flake8
- exit code: 1

tests/apps/test_paint_annotations.py:137:30: E203 whitespace before ':'

Black is adding extra whitespaces that do not match with flake8 config

@mmeendez8 mmeendez8 added bug Something isn't working CI/CD Continuous Integration / Continuous Deployment labels Jan 27, 2021
@igonro igonro self-assigned this Feb 2, 2021
@github-actions
Copy link

github-actions bot commented Feb 2, 2021

@igonro
Copy link
Contributor

igonro commented Feb 4, 2021

Closed by #102

@igonro igonro closed this as completed Feb 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working CI/CD Continuous Integration / Continuous Deployment
Development

No branches or pull requests

2 participants