jiayuasu commented on a change in pull request #523:
URL: https://github.com/apache/incubator-sedona/pull/523#discussion_r632840364
##########
File path: core/pom.xml
##########
@@ -60,6 +60,11 @@
</exclusion>
</exclusions>
</dependency>
+ <dependency>
Review comment:
Please move all geotools dependency to the parent pom. See here:
https://github.com/apache/incubator-sedona/blob/master/pom.xml#L120
Make sure you use the geotools scope variable for the scope
##########
File path: pom.xml
##########
@@ -112,6 +112,13 @@
<version>0.1.0</version>
<scope>${dependency.scope}</scope>
</dependency>
+ <dependency>
+ <groupId>org.geotools</groupId>
+ <artifactId>gt-geotiff</artifactId>
+ <version>${geotools.version}</version>
Review comment:
Same here
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
