Skip to content

Conversation

@dgw
Copy link
Member

@dgw dgw commented Dec 25, 2021

Description

These four functions were for backward compatibility, as they all stand in for features removed in Python 3. Sopel no longer supports Python 2, and it's long past time for plugin authors to use the correct Python 3 functions instead of these.

Honestly, no one should be using these any more in almost-2022. Think of the deprecation warning(s) as a wake-up call to modernize your code!

Checklist

  • I have read CONTRIBUTING.md
  • I can and do license this contribution under the EFLv2
  • No issues are reported by make qa (runs make quality and make test)
  • I have tested the functionality of the things this change touches

Notes

Excluded the now-deprecated module members from coverage reporting, in an attempt to keep our coverage percentage focused only on code we intend to keep. Also excluded these members from the docs. Compare deploy preview to current unstable.

These four functions were for backward compatibility, as they all stand
in for features removed in Python 3. Sopel no longer supports Python 2,
and it's long past time for plugin authors to use the correct Python 3
functions instead of these.

Honestly, no one should be using these any more in almost-2022. Think of
the deprecation warning(s) as a wake-up call to modernize your code!
@dgw dgw added the Housekeeping Code cleanup, removal of deprecated stuff, etc. label Dec 25, 2021
@dgw dgw added this to the 8.0.0 milestone Dec 25, 2021
@dgw dgw requested a review from a team December 25, 2021 18:44
Sphinx will include the `iter*` members in output because they are just
aliases to built-in functions that have docstrings.

Not strictly necessary for `raw_input`, but it's better to be explicit.
@dgw dgw force-pushed the tools-deprecations branch from 9ad2722 to c0ec264 Compare December 25, 2021 18:56
Copy link
Contributor

@Exirel Exirel left a comment

Choose a reason for hiding this comment

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

I like it.

@dgw dgw merged commit d36a19d into master Dec 29, 2021
@dgw dgw deleted the tools-deprecations branch December 29, 2021 20:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Housekeeping Code cleanup, removal of deprecated stuff, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants