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

    https://github.com/apache/incubator-tinkerpop/pull/186#discussion_r49555899
  
    --- Diff: gremlin-core/pom.xml ---
    @@ -37,7 +41,7 @@ limitations under the License.
             <dependency>
                 <groupId>org.yaml</groupId>
                 <artifactId>snakeyaml</artifactId>
    -            <version>1.15</version>
    +            <version>${snakeyaml.version}</version>
    --- End diff --
    
    This version was initially used in the module gremlin-osgi-deps that I just 
removed (see 
https://github.com/gdelafosse/incubator-tinkerpop/commit/f29d01339514837ae4cc98240bc70f53b754aff7).
    So indeed it's no more usefull here, but I usually centralize all the 
version in the dependencyManagment in the parent pom to see them all at a 
glance.
    I'll put it back to its initial location as it's not your rule.


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