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

Full tests for Python2:TokenStreamRewriter, runtime fixes, including fix for #550 (python and java) #1431

Closed
wants to merge 5 commits into from

Conversation

lygav
Copy link
Contributor

@lygav lygav commented Nov 30, 2016

Following #1273

Also fixes #550 for python runtime with new InsertAfter OP type

@lygav lygav closed this Nov 30, 2016
@lygav lygav reopened this Nov 30, 2016
@lygav
Copy link
Contributor Author

lygav commented Dec 1, 2016

Added fix for #550 for Java also

@lygav lygav changed the title Extensive tests for TokenStreamRewriter for Python runtime and general Runtime fixes Full tests for Python2:TokenStreamRewriter, runtime fixes, including fix for #550 (python and java) Dec 1, 2016
@parrt
Copy link
Member

parrt commented Dec 1, 2016

Hi. it'd be better to have separate PRs for the python and java targets, but I'll take a look. thanks!

@parrt parrt added this to the 4.6 milestone Dec 1, 2016
@lygav
Copy link
Contributor Author

lygav commented Dec 1, 2016 via email

@parrt
Copy link
Member

parrt commented Dec 3, 2016

Separated manually. Pulled in #550 fix with #1445

@parrt
Copy link
Member

parrt commented Dec 3, 2016

@lygav I separated python2 update too in #1446

@parrt
Copy link
Member

parrt commented Dec 3, 2016

I will therefore close this one.

@parrt parrt closed this Dec 3, 2016
@lygav
Copy link
Contributor Author

lygav commented Dec 4, 2016

Thank you very much.

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.

TokenStreamRewriter doesn't preserve overrides order
2 participants