Code formatting with Black

[sijanonly]

Code formatting with Black

[sijanonly]

@LucasSD , Plan was to use both together. Even black do most of the formatting we still need flake8 for checking some cases; If I am not wrong.

[sijanonly]

I am working on it.

[LucasSD]

Given we have the workflow pythonpackage.yml using flake8, is this issue now redundant, or do we want to use Black instead of flake8? Or do we want to use both?!

[chriswedgwood]

@sijanonly @LucasSD could we add a .pre-commit-config.yaml. I use cookicutter-django quite a lot and it has a pretty tidy setup for this:

exclude: 'docs|node_modules|migrations|.git|.tox'
default_stages: [commit]
fail_fast: true

repos:

• repo: https://github.com/pre-commit/pre-commit-hooks
  rev: v4.0.1
  hooks:

  • id: trailing-whitespace
  • id: end-of-file-fixer
  • id: check-yaml

• repo: https://github.com/psf/black
  rev: 21.7b0
  hooks:

  • id: black

• repo: https://github.com/timothycrosley/isort
  rev: 5.9.3
  hooks:

  • id: isort

• repo: https://gitlab.com/pycqa/flake8
  rev: 3.9.2
  hooks:

  • id: flake8
    args: ['--config=setup.cfg']
    additional_dependencies: [flake8-isort]

sets up .pre-commit-ci.yaml to ensure pre-commit dependencies stay up to date

ci:
autoupdate_schedule: weekly
skip: []
submodules: false

[sijanonly]

@sijanonly @LucasSD could we add a .pre-commit-config.yaml. I use cookicutter-django quite a lot and it has a pretty tidy setup for this:

exclude: 'docs|node_modules|migrations|.git|.tox'
default_stages: [commit]
fail_fast: true

repos:

* repo: https://github.com/pre-commit/pre-commit-hooks
  rev: v4.0.1
  hooks:
  
  * id: trailing-whitespace
  * id: end-of-file-fixer
  * id: check-yaml

* repo: https://github.com/psf/black
  rev: 21.7b0
  hooks:
  
  * id: black

* repo: https://github.com/timothycrosley/isort
  rev: 5.9.3
  hooks:
  
  * id: isort

* repo: https://gitlab.com/pycqa/flake8
  rev: 3.9.2
  hooks:
  
  * id: flake8
    args: ['--config=setup.cfg']
    additional_dependencies: [flake8-isort]

sets up .pre-commit-ci.yaml to ensure pre-commit dependencies stay up to date

ci:
autoupdate_schedule: weekly
skip: []
submodules: false

Yes. Definitely. We can do that. Let me have a look right away.