-
Notifications
You must be signed in to change notification settings - Fork 45
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
Fix non-empty meta when reading from shapefiles #305
Conversation
I added tests for the other reader types to be safe but I can pull them out if needed. |
Could someone review this? |
Sorry for delays! This looks good. Can you also add a test that verifies that the erroneous line you had in #302 indeed works now? |
Sure! |
I added a test module for IO integration tests. Let me know if this is overkill. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fine with me, thanks!
Actually, the minimal env does not have pyogrio. Can you add skipif to avoid the failure we see now? |
Fixed |
Closes #302