Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use collections.abc.Sequence as the alias is set to be removed #581

Merged
merged 1 commit into from
Nov 29, 2021

Conversation

LordAro
Copy link
Contributor

@LordAro LordAro commented Nov 10, 2021

Alias has been in place since Py 3.3, and minimum (test) version is 3.8, so I think we're safe

Figured I'd save everyone some time and PR it myself

Closes #580

(I'd have done 579 as well, but need your sign off on my proposed hackworkaround to do it)

@pmderodat
Copy link
Member

Hello @LordAro,

Thank you for your contribution! Your patch looks good to me; before we merge it, could you replace “3.8” with “3.7” in the commit message (we target 3.7+) and wrap that message to fit in 76 columns?

Alias has been in place since Py 3.3, and minimum (test) version is 3.7.
@LordAro
Copy link
Contributor Author

LordAro commented Nov 29, 2021

Bleh, you made me actually check out the branch :p Done.

(If you squash the PR, GH UI allows you to reword commits)

@pmderodat
Copy link
Member

Ah, I did not realize that, but I prefered to check with you first anyway. Thanks!

@pmderodat pmderodat merged commit cdc5768 into AdaCore:master Nov 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Direct use of collections breaks with Python3.10
2 participants