Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added in GeoShape Query #9

Closed
wants to merge 2 commits into from
Closed

Added in GeoShape Query #9

wants to merge 2 commits into from

Conversation

byt3sage
Copy link

@byt3sage byt3sage commented Feb 2, 2022

Hi,
I've added in a GeoShape query with a testcase. Also, I'd like to look at maintaining this package if that is possible?

@erichard
Copy link
Owner

erichard commented Feb 2, 2022

Hi,

Thanks for your PR.

I'm more than welcome to others PR if you have others needs.

@byt3sage
Copy link
Author

byt3sage commented Feb 2, 2022

Perfect - I'm just going to bin off the constructor within this class actually - will commit in a minute or two

@byt3sage
Copy link
Author

byt3sage commented Feb 2, 2022

That actually works now 😆

@byt3sage
Copy link
Author

byt3sage commented Feb 2, 2022

Perfect - I'm just going to bin off the constructor within this class actually - will commit in a minute or two

So in relation to this - I removed it temporarily until we decide on how to move forward. I've commented on #7

@byt3sage
Copy link
Author

byt3sage commented Feb 2, 2022

Closing due to #11

@byt3sage byt3sage closed this Feb 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants