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

Remove no-op __future__ imports #39105

Merged
merged 1 commit into from
Mar 21, 2023

Conversation

foolip
Copy link
Member

@foolip foolip commented Mar 21, 2023

These are all no-op (mandatory) since Python 3.0:
https://docs.python.org/3/library/__future__.html

@foolip
Copy link
Member Author

foolip commented Mar 21, 2023

Lots of webdriver tests were affected and run in wpt-chrome-dev-stability, which failed. I'm confident I didn't break it however because this type of change is very unlikely to introduce flakiness, if a mistake is made it's more likely a syntax error or an exception that's always thrown at the same place. Admin merging.

@foolip foolip merged commit 9ae9923 into web-platform-tests:master Mar 21, 2023
@foolip foolip deleted the the-future-is-here branch March 21, 2023 14:40
marcoscaceres pushed a commit that referenced this pull request Mar 28, 2023
cookiecrook pushed a commit to cookiecrook/wpt that referenced this pull request Mar 29, 2023
cookiecrook pushed a commit to cookiecrook/wpt that referenced this pull request Apr 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants