Skip to content

Delete from __future__ import annotations since it requires Python …#11889

Merged
kparzysz-quic merged 1 commit intoapache:mainfrom
kparzysz-quic:no-future-annotations
Jun 24, 2022
Merged

Delete from __future__ import annotations since it requires Python …#11889
kparzysz-quic merged 1 commit intoapache:mainfrom
kparzysz-quic:no-future-annotations

Conversation

@kparzysz-quic
Copy link
Contributor

…3.7+

Copy link
Member

@junrushao junrushao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@zxybazh zxybazh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks for reporting this issue!

@junrushao
Copy link
Member

We should never include python 3.7 features in python 3.6. Just curious why the CI slipped it through :-(

@kparzysz-quic kparzysz-quic merged commit d439f6c into apache:main Jun 24, 2022
@kparzysz-quic kparzysz-quic deleted the no-future-annotations branch June 24, 2022 21:06
@kparzysz-quic
Copy link
Contributor Author

kparzysz-quic commented Jun 24, 2022

I think python 3.6.9 is the default on Ubuntu 18.04, so it would be a bit surprising the CI was using 3.7+.

zxybazh pushed a commit to zxybazh/tvm that referenced this pull request Jun 26, 2022
blackkker pushed a commit to blackkker/tvm that referenced this pull request Jul 7, 2022
mikeseven pushed a commit to mikeseven/tvm that referenced this pull request Sep 27, 2023
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.

3 participants