Github user spmallette commented on a diff in the pull request:

    https://github.com/apache/tinkerpop/pull/499#discussion_r88705921
  
    --- Diff: CHANGELOG.asciidoc ---
    @@ -26,6 +26,12 @@ 
image::https://raw.githubusercontent.com/apache/tinkerpop/master/docs/static/ima
     TinkerPop 3.2.4 (Release Date: NOT OFFICIALLY RELEASED YET)
     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     
    +* Added "attachment requisite" `VertexProperty.element()` and 
`Property.element()` data in GraphSON serialization.
    +* Added `Vertex`, `Edge`, `VertexProperty`, and `Property` serializers to 
Gremlin-Python and exposed tests that use graph object arguments.
    +* `Bytecode.getSourceInstructions()` and `Bytecode.getStepInstructions()` 
now returns `List<Instruction>` instead of `Iterable<Instruction>`.
    +* Added various `TraversalStrategy` registrations with `GryoMapper`.
    +* Fixed a naming mistake in Gremlin-Python: `IdentityRemoveStrategy` is 
now called `IdentityRemovalStrategy`. (*breaking*)
    --- End diff --
    
    I've noticed that in the last few versions you've started adding "breaking" 
to changelog entries. I've been tying "breaking" to tickets that will then show 
up in the changelog on release.  We can start doing this from now i guess, but 
it's not been done consistently in changelog in any way. I guess that's a 
really minor breaking change which wouldn't warrant a separate ticket. I dunno 
- do we want to keep doing this and should it be done consistently going 
forward?


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