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

Allow dbf 5.0+ as dependency #53

Merged
merged 2 commits into from
May 13, 2024
Merged

Allow dbf 5.0+ as dependency #53

merged 2 commits into from
May 13, 2024

Conversation

ilvez
Copy link

@ilvez ilvez commented Apr 26, 2024

dbf released version 5.0.0 (and 5.0.1), which offer support for UTF-8 encoded columns.

I made dbf version required more loose, since 5.0 does not break any compatibility. I'm not library developer, so I don't know whether it's better to keep dependency versions more restricted or looser. In this case dbf 5.0 does not break compatibility in any way.

@BuonOmo
Copy link
Member

BuonOmo commented Apr 26, 2024

I would limit to 5.X though, 6 may break something in the future. WDYT?

Besides that, I'm ok with it seeing the changelog. @keithdoggett opinion?

@ilvez
Copy link
Author

ilvez commented Apr 27, 2024

I would limit to 5.X though, 6 may break something in the future. WDYT?

I updated to not include major after 5.

Besides that, I'm ok with it seeing the changelog. @keithdoggett opinion?

Thanks, I was skimming through files and didn't see CHANGELOG. Updated History.md now.

@ilvez
Copy link
Author

ilvez commented May 10, 2024

Can I help somehow to get this merged?

@BuonOmo
Copy link
Member

BuonOmo commented May 10, 2024

@ilvez I don't think. @keithdoggett should review it and we'll be set :)

Copy link
Member

@keithdoggett keithdoggett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM sorry about the delay

@keithdoggett keithdoggett merged commit fad1969 into rgeo:main May 13, 2024
12 of 13 checks passed
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.

3 participants