Skip to content

Conversation

@Kilo59
Copy link
Collaborator

@Kilo59 Kilo59 commented Mar 25, 2020

Addresses #187
Will also impact #169

  • Configured black and isort to play nice with each other and respect a line length of 120.
  • added make fmt
  • added make check-fmt

https://github.com/psf/black
https://github.com/timothycrosley/isort

TODO

  • configure pylint for line-length 120
  • configure pylint for formatting rules

@Kilo59 Kilo59 requested a review from ExpDev07 March 25, 2020 22:09
# Mapping of country names to alpha-2 codes according to
# https://en.wikipedia.org/wiki/ISO_3166-1.
# As a reference see also https://github.com/TakahikoKawasaki/nv-i18n (in Java)
# fmt: off
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I turned the formatting off here since it looks so nice 😄

Comment on lines 23 to 29
fmt:
isort -rc --atomic
black .

check-fmt:
isort -rc --check
black . --check --diff
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

new fmt and check-fmt commands

Kilo59 and others added 2 commits March 25, 2020 19:57
disable `bad-continuation`
up max-line-length to `120`
Copy link
Owner

@ExpDev07 ExpDev07 left a comment

Choose a reason for hiding this comment

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

Looks good!

Copy link
Owner

@ExpDev07 ExpDev07 left a comment

Choose a reason for hiding this comment

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

nicely done. LGTM.

@ExpDev07 ExpDev07 merged commit 8678f84 into ExpDev07:master Mar 26, 2020
@ExpDev07 ExpDev07 linked an issue Mar 26, 2020 that may be closed by this pull request
@Kilo59 Kilo59 deleted the format branch March 29, 2020 23:35
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.

Adopting a formatter and/or import sorter

2 participants