#50118 Issue 50032 - Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated in Python 3.7
Closed by spichugi. Opened by vashirov.
vashirov/389-ds-base fix_deprecation_warnings  into  master

Download 50118.patch

Bug Description:
Deprecation warnings are issued by Python for the following changes:
1. https://docs.python.org/3/whatsnew/3.6.html#deprecated-python-behavior
https://bugs.python.org/issue27364 - Deprecate invalid escape sequences in str/bytes
2. https://docs.python.org/3/whatsnew/3.7.html#deprecated-python-behavior
https://bugs.python.org/issue25988 - collections.abc.Indexable
3. https://docs.python.org/3/library/logging.html#logging.warning
https://bugs.python.org/issue13235 - logging.warn() is not documented

Fix Description:
1. Use correct escape sequences or raw strings where needed.
2. Import Callable from collections.abc instead of collections module directly.
3. Use logging.warning() instead of logging.warn().

https://pagure.io/389-ds-base/issue/50032

Reviewed by: ???

Just a small nitpick - the commit subject is too long.
The rest looks good to me.

Hey @vashirov I think this can be merged?

rebased onto 7a0b8ae5fcb99d5948e771b04cb730650094cb47

Pull-Request has been merged by vashirov

389-ds-base is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in 389-ds-base's github repository.

This pull request has been cloned to Github as issue and is available here:
- https://github.com/389ds/389-ds-base/issues/3177

If you want to continue to work on the PR, please navigate to the github issue,
download the patch from the attachments and file a new pull request.

Thank you for understanding. We apologize for all inconvenience.

Pull-Request has been closed by spichugi

Metadata