You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The reason will be displayed to describe this comment to others. Learn more.
Apologies, I was looking at the change long and I believe I entered #421 into the wrong section in the changelog as it's currently sitting under 0.8.0.
Should I raise a PR that would then get cherry-picked to v0.9.0?
The reason will be displayed to describe this comment to others. Learn more.
Ah, good catch .. Doing a PR to fix this on main would be good, yes (at the moment we don't have a 0.9.0 branch to cherry-pick to, so just main is fine)
568f870
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.
Apologies, I was looking at the change long and I believe I entered #421 into the wrong section in the changelog as it's currently sitting under 0.8.0.
Should I raise a PR that would then get cherry-picked to v0.9.0?
568f870
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.
Ah, good catch .. Doing a PR to fix this on main would be good, yes (at the moment we don't have a 0.9.0 branch to cherry-pick to, so just
main
is fine)568f870
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.
I already updated the notes in the github release: https://github.com/geopandas/pyogrio/releases/tag/v0.9.0
568f870
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.
Thanks @jorisvandenbossche , I've raised #429 to correct the error in the changelog