petern48 opened a new issue, #289:
URL: https://github.com/apache/sedona-db/issues/289

   I notice sometimes people create PRs from their `main` branch. We should 
recommend to create a separate one in the [contributor 
guide](https://github.com/apache/sedona-db/blob/94ce7db5287b39218b803eae057a4eb92a50c397/docs/contributors-guide.md).
 Plus add a small note on why it's generally recommended (which is stuff that's 
not specific to Sedona, but just in GitHub in general
   - keeping `main` clean allows you to merge upstream main into your branch if 
needed (e.g merge conflicts)
   - you can open separate PRs in separate branches, instead of having them 
overwrite each other. I've seen people overwrite and delete their code after 
pulling in the latest main to their fork
   
   There are instructions for spinning up the docker container for python 
testing purposes, but not how to query PostGIS shell directly. In nearly all 
cases, this command should work to enter the PostGIS shell.
   
   ```
   docker exec -it sedona-db-postgis-1 psql -U postgres
   ```


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