jiayuasu commented on PR #843:
URL: https://github.com/apache/sedona/pull/843#issuecomment-1565138230

   @kartikeyhadiya Thanks for your contribution. A few comments:
   
   1. Since this is a major contribution, please create a JIRA ticket first.
   2. Please do not duplicate any data files in the source repo. The source 
repo already has a couple of data files in `binder`. Please try to reuse those 
via Dockerfile.
   3. Please do not directly copy jars to the source repo. Please download jars 
on the fly via Dockerfile.
   4. All source code such as .sh script must have Apache License as the file 
header.
   5. Once everything is done, you will need to add the docker compilation to 
GitHub Action with a Yaml file.


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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to