No need to have future as dependency #115
Comments
|
You are probably right. Away on holidays at the moment l, will take a look
when backm
…On Wed, 12 July 2023, 9:31 pm frenzymadness, ***@***.***> wrote:
future is listed in dependencies but I cannot find a usage of it in the
project and therefore I think it can be removed from the list. future is
broken with Python 3.12 and will be even more with 3.13.
https://github.com/google/textfsm/blob/a4c2f824eb53d32ccae810068867a58cb9be17fe/setup.py#L56
—
Reply to this email directly, view it on GitHub
<#115>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADD2VEQASGCJBFBNEERPWGLXP2RK3ANCNFSM6AAAAAA2HP54NY>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
This was referenced Aug 14, 2023
|
I see, thank you. |
|
Opened issue #119 to clean the legacy code
…On Sun, 17 Sept 2023 at 22:52, frenzymadness ***@***.***> wrote:
I see, thank you.
—
Reply to this email directly, view it on GitHub
<#115 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADD2VEUX6F35SEETB3DEKWLX23XBBANCNFSM6AAAAAA2HP54NY>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
|
#121 This change is in now |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
futureis listed in dependencies but I cannot find a usage of it in the project and therefore I think it can be removed from the list.futureis broken with Python 3.12 and will be even more with 3.13.textfsm/setup.py
Line 56 in a4c2f82
The text was updated successfully, but these errors were encountered: