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

Revert to previous merge method #496

Merged
merged 1 commit into from
Apr 29, 2017

Conversation

JordanMartinez
Copy link
Contributor

@JordanMartinez JordanMartinez commented Apr 28, 2017

Reverts back to the previous merge method

@TomasMikula
Copy link
Member

The type param on UndoActions should not be necessary.

@JordanMartinez
Copy link
Contributor Author

As said in my other comment, should the TextChange class-to-interface conversion be done at all? If it should, I feel like it should be in its own PR that does a lot of other refactoring.

@TomasMikula
Copy link
Member

Yeah, not in this PR. Not sure what it gives us anyway.

@JordanMartinez
Copy link
Contributor Author

Not really anything. It was only necessary because of my error in adding a type param to UndoActions

I'll fix this tomorrow.

@JordanMartinez JordanMartinez changed the title Revert to previous merge method; add type param to UndoActions Revert to previous merge method Apr 29, 2017
@JordanMartinez
Copy link
Contributor Author

Ok. Only the revert is in this PR now.

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.

None yet

2 participants