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

bpo-33487: improve BZ2File Deprecation and documentation. #6785

Merged
merged 1 commit into from Sep 11, 2018

Conversation

Carreau
Copy link
Contributor

@Carreau Carreau commented May 13, 2018

Emit warning when None passed explicitly, list Python version since
deprecation in warning message and docs.

https://bugs.python.org/issue33487

Copy link
Member

@matrixise matrixise left a comment

Choose a reason for hiding this comment

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

could you fix the NEWS entry, thank you

@@ -0,0 +1,3 @@
BZ2file now emit a DeprecationWarning when buffering=None is passed, the
deprecation mesage and documentation also now explicitely state it is
Copy link
Member

Choose a reason for hiding this comment

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

there is a typo in your "message".

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oops, thanks. Fixed.

Emit warning when None passed explicitly, list Python version since
deprecation in warning message and docs.
@Carreau
Copy link
Contributor Author

Carreau commented May 16, 2018

Updated version since deprecation as apparently it's deprecated since 3.0.

@gpshead gpshead merged commit ffa198c into python:master Sep 11, 2018
@Carreau Carreau deleted the bz2file-deprecation branch September 11, 2018 07:36
@Carreau
Copy link
Contributor Author

Carreau commented Sep 11, 2018

Thanks @matrixise amd @gpshead.

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.

None yet

5 participants