Skip to content

Conversation

JelleZijlstra
Copy link
Member

@JelleZijlstra JelleZijlstra commented Mar 18, 2018

This reverts the lib2to3 changes in commit ac31770. As @ambv pointed out in #1669 (comment), this change is unnecessary and broke usage of 2to3 for pre-3.7 code.

This should be backported to 3.7 because it fixes a regression that was introduced by ac31770.

https://bugs.python.org/issue30406

This reverts commit ac31770.

(Reverts only the lib2to3 part.)
@ambv
Copy link
Contributor

ambv commented Mar 18, 2018

Jelle, thanks a ton for working on this! I will merge as soon as tests pass.

@ambv ambv merged commit f64aae4 into python:master Mar 18, 2018
@miss-islington
Copy link
Contributor

Thanks @JelleZijlstra for the PR, and @ambv for merging it 🌮🎉.. I'm working now to backport this PR to: 3.7.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Mar 18, 2018
…" (pythonGH-6143)

This reverts commit ac31770.

(Reverts only the lib2to3 part.)
(cherry picked from commit f64aae4)

Co-authored-by: Jelle Zijlstra <jelle.zijlstra@gmail.com>
@bedevere-bot
Copy link

GH-6145 is a backport of this pull request to the 3.7 branch.

miss-islington added a commit that referenced this pull request Mar 18, 2018
…H-6143)

This reverts commit ac31770.

(Reverts only the lib2to3 part.)
(cherry picked from commit f64aae4)

Co-authored-by: Jelle Zijlstra <jelle.zijlstra@gmail.com>
jo2y pushed a commit to jo2y/cpython that referenced this pull request Mar 23, 2018
@JelleZijlstra JelleZijlstra deleted the fix2to3async branch May 26, 2023 06:07
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.

5 participants