Github user spmallette commented on the issue:

    https://github.com/apache/tinkerpop/pull/501
  
    @joshsh - thanks - note that we still support 3.1.x (mostly for 
maintenance) on the tp31 branch, 3.2.x on the tp32 branch (for maintenance and 
non-breaking new features) and then do all dev for the future 3.3.x on master 
(which is all new stuff including breaking changes). In this case, I don't 
think it would hurt to put this on tp31 as @okram suggests - it might be a 
useful little bit of functionality for someone still on  3.1.x and shouldn't 
cause any conflicts. 
    
    Is there an easy test you could write that just exercises the XSLT a little 
bit? Perhaps you could just put something in `gremlin-test` in `IoTest` as we 
have tests in there like `shouldReadLegacyGraphSON` which would be similar to 
what you have here. Maybe just copy that model:
    
    
https://github.com/apache/tinkerpop/blob/492c8e160743bb6d15bc07262a47275d18242eed/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/structure/io/IoTest.java#L438-L452
    
    and use your XSLT instead of the `LegacyGraphSONReader`?


---
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.
---

Reply via email to