Github user jorgebay commented on the issue:
https://github.com/apache/tinkerpop/pull/450
I've added some fixes during the past weeks.
I'll try to summarize the open issues with this patch to try to unblock it:
A) Currently, the javascript GLV exposes `list()` instead of `toList()`. As
it takes a callback as a parameter, having the method start with a `toX()`
didn't feel natural to me.
Maybe it makes more sense to follow the existing naming (`toList()`).
Exposing something uniform across all GLVs, whenever possible, can be more
important (in the same way as Python GLV used mixedCase to be more close to
gremlin).
The same applies to `next()`. To avoid confusions, we could document that
`traversal.next()` is not an iterator and returns the first result.
B) Promise-based API.
We can include a promise-based method overloads when supported by the js
engine, like [proposed above][1]. We can do it now, before merging it or as a
next step after merging.
Can we try to reach a conclusion on these issues?
After that I can send a draft documentation for `gremlin-variants.asciidoc`.
I've been using the Javascript GLV for a while now and it feels good to
have code completion for Traversal methods on the IDE!
[1]: https://github.com/apache/tinkerpop/pull/450#issuecomment-252911095
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
