Skip to content

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