Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Black #29

Merged
merged 3 commits into from
Jul 25, 2019
Merged

Black #29

merged 3 commits into from
Jul 25, 2019

Conversation

Um9i
Copy link
Contributor

@Um9i Um9i commented Jul 25, 2019

#26

@hackebrot
Copy link
Owner

Hi @Um9i! 👋

Thank you for this pull-request. I'd rather not remove flake8 from our test environments. Would you mind reverting that specific change?

@hackebrot hackebrot added the enhancement Enhancement proposal for poyo label Jul 25, 2019
@Um9i
Copy link
Contributor Author

Um9i commented Jul 25, 2019

this is fixable with a rerun of black with the --line-length parameter if that's the preferred way of going about it.

@hackebrot hackebrot merged commit 1a0983d into hackebrot:master Jul 25, 2019
@hackebrot
Copy link
Owner

I just updated the flake8 max line length to the black default line length value.

See https://github.com/hackebrot/poyo/blob/master/setup.cfg#L17

Thanks again @Um9i! 🐓

@Um9i Um9i deleted the black branch July 25, 2019 11:31
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement Enhancement proposal for poyo
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants