Github user aholmberg commented on the pull request:

    
https://github.com/apache/tinkerpop/commit/6ed7edc0b4ad2abf933e917812d49ad92230c8d1#commitcomment-18473940
  
    In 
gremlin-variant/src/main/groovy/org/apache/tinkerpop/gremlin/python/GremlinPythonGenerator.groovy:
    In 
gremlin-variant/src/main/groovy/org/apache/tinkerpop/gremlin/python/GremlinPythonGenerator.groovy
 on line 95:
    May I suggest that we use a trailing, instead of leading underscore for 
keyword collisions? This would make it more natural to begin typing for 
completion. It would also adhere to Python style guidelines, which says leading 
underscore indicates private, internal use, and to use trailing underscores to 
avoid keyword collisions.
    https://www.python.org/dev/peps/pep-0008/#descriptive-naming-styles


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to