You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@tinkerpop.apache.org by justinchuch <gi...@git.apache.org> on 2018/09/24 16:52:30 UTC

[GitHub] tinkerpop pull request #936: Apply StringEscapeUtils in GroovyTranslator#con...

GitHub user justinchuch opened a pull request:

    https://github.com/apache/tinkerpop/pull/936

    Apply StringEscapeUtils in GroovyTranslator#convertToString(String) [tp33]

    Preface:
    There's character escaping issue when translating.
    
    What's changed:
    - Merge the changes from tp32
    
    - org.apache.tinkerpop.gremlin.groovy.jsr223.GroovyTranslatorTest
    Adjust the test case to verify the changes in tp33.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/justinchuch/tinkerpop groovy-translator-tp33

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/tinkerpop/pull/936.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #936
    
----
commit aff2037b5973c62c0292ab3065761e923899aa57
Author: Justin Chu <15...@...>
Date:   2018-09-21T19:12:26Z

    Apply StringEscapeUtils in GroovyTranslator#convertToString(String)

commit 02e412d92d6f6c7004bdeef206e3d7adb9bd148a
Author: Justin Chu <15...@...>
Date:   2018-09-24T15:41:36Z

    Merge branch 'groovy-translator-tp32' into groovy-translator-tp33

commit 3f221968d79e2db6e170eec9285cb75d8cb203f0
Author: Justin Chu <15...@...>
Date:   2018-09-24T16:44:17Z

    Adjust test cases for tp33

----


---

[GitHub] tinkerpop pull request #936: Apply StringEscapeUtils in GroovyTranslator#con...

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

    https://github.com/apache/tinkerpop/pull/936


---