Breaking change to configuration logic in v2.1.0 about autoflake HOT 8 OPEN

In version 2.1.1, I'm seeing that [autoflake] works, but [tool.autoflake] does not for a pyproject.toml file.

This is in direct contradiction to the PyPi docs

from autoflake.

@joeyagreco ahh ok, yes --config doesn't support pyproject.toml at the moment. It's being parsed as an ini file, and toml just happens to be compatible with that.

from autoflake.

Hi @kevinr-electric, thanks for reporting. I'm going to revert that change and then we can have a separate flag that's exclusive for the location of pyproject.toml.

Currently, autoflake doesn't keep a changelog in a dedicated location. We can introduce something with GitHub releases though.

from autoflake.

Hey @fsouza, thanks for the update. A releases section with release notes would also be appreciated!

from autoflake.

I'm confused here; Passing a toml file to --config was completely broken before (that was the whole point of this PR). How were you doing that?

from autoflake.

@joeyagreco can you share an example? I can't repro it:

％ cat pyproject.toml
[project]
name = "sample"
description = "Just a sample"
license = { text = "ISC" }

[tool.autoflake]
expand-star-imports = true
％ cat sample.py
import os
from sys import *

print("hi", file=stderr)
％ autoflake sample.py
--- original/sample.py
+++ fixed/sample.py
@@ -1,4 +1,3 @@
-import os
-from sys import *
+from sys import stderr

 print("hi", file=stderr)
％ autoflake --version
autoflake 2.1.1

from autoflake.

Sure! @fsouza

Here's my pyproject.toml file

[autoflake]
ignore_init_module_imports = true
in_place = true
recursive = true
remove_all_unused_imports = true

Check version

% autoflake --version
> autoflake 2.1.1

Run it

% autoflake --config=pyproject.toml .
> 

Works (removes an unused import)

Let's change the pyproject.toml

[tool.autoflake]
ignore_init_module_imports = true
in_place = true
recursive = true
remove_all_unused_imports = true

Now let's run it again

% autoflake --config=pyproject.toml .
> can't parse config file 'C:\foo\baz\pyproject.toml'

from autoflake.

I'm confused here; Passing a toml file to --config was completely broken before (that was the whole point of #249). How were you doing that?

Sorry, I had missed this message, but I assume people were relying on the fact that simple toml files can be parsed as ini files. After thinking about this a little bit, I feel like we can roll #249 forward again. Alternatively, we could have a dedicated flag for pyproject.toml.

from autoflake.