-
Notifications
You must be signed in to change notification settings - Fork 175
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
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 |
# for free
to join this conversation on GitHub.
Already have an account?
# to comment
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.textfsm/setup.py
Line 56 in a4c2f82
The text was updated successfully, but these errors were encountered: