[GitHub] [tinkerpop] thefliik commented on issue #1056: fix: Path#toString() variable reference bug

2019-02-06 Thread GitHub
The code in question on `tp33`: https://github.com/apache/tinkerpop/blob/tp33/gremlin-javascript/src/main/javascript/gremlin-javascript/lib/structure/graph.js#L118-L139 [ Full content available at: https://github.com/apache/tinkerpop/pull/1056 ] This message was relayed via gitbox.apache.org for

[GitHub] [tinkerpop] thefliik commented on issue #1056: fix: Path#toString() variable reference bug

2019-02-06 Thread GitHub
@jorgebay in the `tp33` branch it looks like the Path#toString() method has been removed entirely, so when/if that branch is merged into master this PR will be unnecessary. Feel free to close. [ Full content available at: https://github.com/apache/tinkerpop/pull/1056 ] This message was relayed

[GitHub] [tinkerpop] thefliik commented on issue #1056: fix: Path#toString() variable reference bug

2019-02-05 Thread GitHub
Seperately, would you be interested in a PR to port the whole javascript library over to typescript? [ Full content available at: https://github.com/apache/tinkerpop/pull/1056 ] This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org