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

drop deprecated ignore_ext parameter #661

Merged
merged 3 commits into from
Mar 30, 2022
Merged

drop deprecated ignore_ext parameter #661

merged 3 commits into from
Mar 30, 2022

Conversation

mpenkov
Copy link
Collaborator

@mpenkov mpenkov commented Oct 11, 2021

This is a follow up from #607

We currently have two mutually exclusive parameters, ignore_ext (deprecated) and compression. We introduced the new one and marked the old one as deprecated about 6 months ago. I think sufficient time has passed for us to remove this parameter.

We still have a function for compatibility with ancient code (smart_open.smart_open). That's still there and remains the same.

This is a backwards-incompatible change, so we should probably put it in the same release as #660

Copy link
Owner

@piskvorky piskvorky left a comment

Choose a reason for hiding this comment

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

Thanks, looks good. But what's the migration path? What should users with ignore_ext in their code do now?

I checked https://github.com/RaRe-Technologies/smart_open/blob/develop/MIGRATING_FROM_OLDER_VERSIONS.rst and it says only The ignore_extension keyword parameter is now called ignore_ext. It behaves identically otherwise..

@mpenkov
Copy link
Collaborator Author

mpenkov commented Feb 18, 2022

OK, added migration docs. Please have a look.

@piskvorky piskvorky added this to the 6.0.0 milestone Feb 18, 2022
@mpenkov
Copy link
Collaborator Author

mpenkov commented Mar 30, 2022

@piskvorky I'm going to merge this now, and release 6.0.0 within the next several days. Please have a look and let me know if anything looks fishy.

@mpenkov mpenkov merged commit 84e9aac into develop Mar 30, 2022
@mpenkov mpenkov deleted the rm-ignore-ext branch March 30, 2022 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants