AdventureLookup / adventurelookup-backend

Platform, Framework, Database software
GNU General Public License v3.0
28 stars 7 forks source link

Rename [style] section of setup.cfg to [yapf] #24

Closed whonut closed 8 years ago

whonut commented 8 years ago

yapf settings should be in the [style] section of a .style.yapf file, but the [yapf] section of a setup.cfg. From the docs (emphasis mine):

YAPF will search for the formatting style in the following manner:

  1. Specified on the command line
  2. In the [style] section of a .style.yapf file in either the current directory or one of its parent directories.
  3. In the [yapf] secionf (sic) of a setup.cfg file in either the current directory or one of its parent directories.
  4. In the ~/.config/yapf/style file in your home directory.

yapf ignored altered max line length until I changed the section name.

This almost seems unworthy of a PR, but hey-ho.

pejter commented 8 years ago

My bad, thanks for fixing :+1: And the issue would be just as good, so it doesn't matter if you make a PR