Closed Just-gomin closed 1 year ago
Oh hi @Just-gomin, thanks for the addition, but the merge base branch should be main
.
Sorry, I only just saw the PR.
Thanks for your contribution! 😄
Thank you for your answer.
It was my first contribution, so I missed the 'merge base branch should be main
' part.
I have created a new 'Pull Request' based on your advice.
I'd appreciate it if you could check it.😄
Thanks!
Yes, I will check.
Also, I'm closing this pull request in favor of #145.
Add missing parameter 'unit' when calling "_nearestPointOnLine" in function "nearestPointOnLine".