macisamuele / language-formatters-pre-commit-hooks

Collection of custom pre-commit hooks.
Apache License 2.0
116 stars 58 forks source link

pretty-format-ini broken in v2.0.0 #99

Closed lcnittl closed 2 years ago

lcnittl commented 2 years ago

It seems that since the ini formatter is broken since including v2.0.0.

This file for example:

[   flake8  ]

max-line-length    =         88

extend-ignore          =
                                         E203

passes, not being reformatted. In v1.6.1 this file was reformatted to:

[   flake8  ]
max-line-length = 88
extend-ignore =
    E203

Cheers, lcnittl

macisamuele commented 2 years ago

@lcnittl thanks for the report

I think that the regression was introduced in #45 I'm not yet 100% sure on the path forward, probably iniparse library has some toggle that we might use a different parser as suggested in #26 Need to verify a bit more

macisamuele commented 2 years ago

Fixed in #113