[GitHub] tinkerpop issue #852: TINKERPOP-1944 Export DriverRemoteConnection

2018-04-26 Thread jbmusso
Github user jbmusso commented on the issue: https://github.com/apache/tinkerpop/pull/852 VOTE +1 ---

[GitHub] tinkerpop issue #695: TINKERPOP-1489 JavaScript GLV

2018-01-23 Thread jbmusso
Github user jbmusso commented on the issue: https://github.com/apache/tinkerpop/pull/695 All good to me! VOTE +1 ---

[GitHub] tinkerpop issue #695: TINKERPOP-1489 JavaScript GLV

2018-01-19 Thread jbmusso
Github user jbmusso commented on the issue: https://github.com/apache/tinkerpop/pull/695 According to https://github.com/petkaantonov/bluebird/issues/1026, users should be able to just patch the global `Promise` object in their application with: ```javascript global.Promise

[GitHub] tinkerpop issue #695: TINKERPOP-1489 JavaScript GLV

2018-01-17 Thread jbmusso
Github user jbmusso commented on the issue: https://github.com/apache/tinkerpop/pull/695 Finally found some time. Wew. Well, this PR is very well crafted - well done! My comments are only minor: * About `utils.toPromise` - if I understand you well, you want a dual

[GitHub] tinkerpop issue #695: TINKERPOP-1489 JavaScript GLV

2018-01-11 Thread jbmusso
Github user jbmusso commented on the issue: https://github.com/apache/tinkerpop/pull/695 Quick update - I plan to check this PR this weekend. ---

[GitHub] tinkerpop issue #450: TINKERPOP-1489 Javascript GLV

2016-10-08 Thread jbmusso
Github user jbmusso commented on the issue: https://github.com/apache/tinkerpop/pull/450 I was thinking that supporting JavaScript `Promise` could be helpful now that they’ve been part of the ES2015 and rolled out on most JS environment (browsers/Node.js). Maybe this can