Add black as a code formatter about ynab-sdk-python HOT 8 CLOSED

Is it possible to use black formatter with single quotes ? (It's my preference)

from ynab-sdk-python.

You can use --skip-string-normalization option to avoid converting single quotes to double quotes

from ynab-sdk-python.

Ok, I'm still undecided about single/double quotes, but I agree that black can be a good addition to this project.

I have zero knowledge abou github actions, can we mix them with travis ?

Also, do you want to tackle this one ?

And last but not least, add the black badge on the readme 🏴

from ynab-sdk-python.

We can use pre-commit. I've found this guide:

https://www.mattlayman.com/blog/2018/python-code-black/

You have to add pre-commit to requirements-dev.txt, create the file .pre-commit-config.yaml in your repo and add to this file the following:

repos:

• repo: https://github.com/ambv/black
  rev: stable
  hooks:
  • id: black
    language_version: python3.7

This way black will check and modify every code before being committed.
Although i don't know if it can be used with the --skip-string-normalization option

from ynab-sdk-python.

The pre-commit could be an extra, but since it can be skipped, its important to run the check on the CI.

It seems to be easy to do this in the lint step (That currently does nothing 😆 )

from ynab-sdk-python.

ok, so you just have to add "black --check ." to the lint step?

from ynab-sdk-python.

from ynab-sdk-python.

Done by #71

from ynab-sdk-python.