petern48 commented on PR #409:
URL: https://github.com/apache/sedona-db/pull/409#issuecomment-3622889555

   Could you also fix the PR title and mention the issue in the PR description, 
according to my previous comment before: 
https://github.com/apache/sedona-db/pull/386#issuecomment-3591890245. You 
should generally try to do it whenever you first open the PR. If you don't do 
so, it increases the odds of someone else taking up that work too, which would 
eventually lead to someone (either you or them) going through the work just for 
their PR to not be merged.
   
   If you're afraid of linking the issue earlier because you're not sure you 
can 100% commit, it's totally okay to link it anyways and later decide to close 
your PR if you decide you don't want to follow through with it (e.g. if it's 
too hard or you lose interest). Nothing ever locks you in to being *required* 
to finish something. But please do link the issue as soon as you open the PR. 
For this PR, we're already close to being done anyway.


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