[ 
https://issues.apache.org/jira/browse/TINKERPOP-1443?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15703149#comment-15703149
 ] 

ASF GitHub Bot commented on TINKERPOP-1443:
-------------------------------------------

Github user metlos commented on the issue:

    https://github.com/apache/tinkerpop/pull/494
  
    `.travis.install-maven.sh` is a workaround I used in another project for 
https://github.com/travis-ci/travis-ci/issues/4872. Trusty, which is the distro 
used on Travis for Tinkerpop builds, provides Maven 3.1.1, which is too old for 
Revapi which needs at least 3.2.5.
    
    I will push another commit with the changes necessary to track the ignored 
(aka intentional) API changes. It will be similar to 
http://revapi.org/modules/revapi-maven-plugin/examples/multi-file-configuration.html.


> Use an API checker during build
> -------------------------------
>
>                 Key: TINKERPOP-1443
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-1443
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: build-release
>    Affects Versions: 3.2.2
>            Reporter: Lukas Krejci
>
> Tinkerpop 3.2.2 changed the signature of the method 
> {{GraphTraversal.hasLabel}} from {{(String...)}} to {{(String, String...)}}. 
> While this is certainly an improvement, it is both source and binary 
> incompatible change.
> I.e. even if every usage of {{hasLabel}} had at least one parameter in the 
> user code, none of those calls will work until all the user code is 
> recompiled using Tinkerpop 3.2.2.
> I don't know the versioning policy of Tinkerpop but changes like the above in 
> a micro/patch release are generally unexpected.
> Please consider API checkers like http://revapi.org to warn about such 
> incompatible API changes...



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to