Skip to content

Conversation

ki4070ma
Copy link
Collaborator

@ki4070ma ki4070ma commented Jan 24, 2021

Background

We don't need to pay attention to code format anymore.

Please see detailed concept here.
https://github.com/psf/black

Any comments are welcome.

Tests

multi_line_output = 3
known_third_party = dateutil,httpretty,pytest,selenium,setuptools,urllib3,mock,sauceclient
known_first_party = test,appium
multi_line_output = 3
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mykola-mokhnach
Copy link
Contributor

I like how it formats the code now

@mykola-mokhnach mykola-mokhnach changed the title [DO NOT MERGE] Applied black [DO NOT MERGE] chore: Apply Black code formatter Jan 26, 2021
@ki4070ma ki4070ma changed the title [DO NOT MERGE] chore: Apply Black code formatter chore: Apply Black code formatter Jan 28, 2021
@ki4070ma ki4070ma marked this pull request as ready for review January 28, 2021 13:21
@ki4070ma ki4070ma requested a review from KazuCocoa January 28, 2021 13:21
Copy link
Member

@KazuCocoa KazuCocoa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

like it :)

@ki4070ma ki4070ma merged commit 344953a into appium:master Jan 29, 2021
@ki4070ma ki4070ma deleted the formatter-black branch January 29, 2021 14:55
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.

3 participants