Santhosh
Santhosh
I signed it!
@NeilFraser I have taken care of comments. About > Also, are all those sections of pom.xml needed? There seems to be a lot of boilerplate copied from some other project....
@NeilFraser I have converted your tests to Junit based tests so that you can just execute them with command `mvn test`, which will take care to setup classpath etc.
@franklinyu Reason nexus related stuff is there with assumption that Neil would want to push this library to Maven Central so that is can be easily adopted by others. However...
1. Removed the nexus related config from pom.xml 2. Removed the conflicts with latest 3. Integrated latest diff_match_patch_test.java to run as Junit tests @NeilFraser ready for another review.
About 1.8: If code is not using any 1.8 features, why make it that version as minimum? About rebasing: Unless Neil confirms that he will review and merge, there is...
@mithuns I'd understand the merits of 1.8. But it is upto Neil to make that call.
fyi that ReadableStream is supported in all evergreen browsers. https://caniuse.com/#search=ReadableStream
While it is not prettier domain to transform the code, I think this is a great feature to have with additional config (turned off by default). This could be done...
Instead of additional attribute, I would think having additional options "semantic" or something to that effect would be more easier to consume. As current options is meant to alter how...