Remove unused import

[sammyhori]

Removed a single unused import.

There also exists an unused import at pyaction/template/{{action_slug}}/main.py but that file only contains the import so I let it be.

[lnxpy]

Thank you @sammyhori! The change that you made in the io.py file should be changed. I've imported Dict but used the built-in dict. Since PyAction supports Python 3.8 all the way up to 3.12, we better use Dict other than the built-in dict.

The pyaction/template/{{action_slug}}/main.py file, it's a template file that even pre-commit ignores its parent directory while checking. Consider it as a blueprint.

[sammyhori]

Thank you @sammyhori! The change that you made in the io.py file should be changed. I've imported Dict but used the built-in dict. Since PyAction supports Python 3.8 all the way up to 3.12, we better use Dict other than the built-in dict.

Fixed in 94bd49e

The pyaction/template/{{action_slug}}/main.py file, it's a template file that even pre-commit ignores its parent directory while checking. Consider it as a blueprint.

👍 All good, I thought it was probably something like that.

[sammyhori]

The pre-commit hook automatically changes Dict to dict, thus undoing my changes.

[lnxpy]

@sammyhori, It's because of the from __future__ import annotations import line in the io.py file. The pyupgrade hook will consider it and change typing.Dict to dict to make the source file be supported on all Python environments. Removing it (the import line) wouldn't make a difference.

Things to do..

• Remove the from __future__ import annotations line from the beginning of the io.py file.
• Convert dict to Dict.
• Squash all your commits into one if you can, please.

Hopefully, it's gonna be alright this time. :))

[sammyhori]

@sammyhori, It's because of the from __future__ import annotations import line in the io.py file. The pyupgrade hook will consider it and change typing.Dict to dict to make the source file be supported on all Python environments. Removing it (the import line) wouldn't make a difference.

Things to do..

• Remove the from __future__ import annotations line from the beginning of the io.py file.
• Convert dict to Dict.
• Squash all your commits into one if you can, please.

Hopefully, it's gonna be alright this time. :))

All fixed this time, thanks for the advice :)

[lnxpy]

Very well done, @sammyhori!

[lnxpy]

Thanks, mate! Hope to see more from you! :D