search

Fueled / django-init

Project template used at Fueled for scaffolding new Django based projects. :dizzy:
Other
189 stars 46 forks source link

fix(*): Changes for black #402

Closed CuriousLearner closed 4 years ago

CuriousLearner commented 4 years ago

Why was this change necessary?

To follow the PEP8 rules enforced by black.

How does it address the problem?

Minimal changes to follow the styles enforced by black. Few of the changes aren't included which made the indentation worse or were related to line-length dependent on the project name

Are there any side effects?

None.

theskumar commented 4 years ago

Few of the changes aren't included which made the indentation worse or were related to line-length dependent on the project name.

This should ideally go in the precommit hooks as well and it must be implemented for all. You can have the line-length changed to 110 which gives the best output along with isort.

CuriousLearner commented 4 years ago

@theskumar Can you please elaborate a bit on this:

This should ideally go in the precommit hooks as well and it must be implemented for all.

tucosaurus commented 4 years ago

This should ideally go in the precommit hooks as well and it must be implemented for all.

I can create a PR for this. Have been doing this for all my projects.