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

Replace SopelWrapper by Sopel in the code, and add code deprecation warning to prevent its future usage #2639

Open
Exirel opened this issue Oct 21, 2024 · 0 comments

Comments

@Exirel
Copy link
Contributor

Exirel commented Oct 21, 2024

Part of the replacement of SopelWrapper with contextvars:

  • Sopel should be used instead of SopelWrapper in the code
  • improve testing tools so it is much easier to test with a "context sensitive instance" to ease future breaking change
  • extensively document how to test with Sopel instead of SopelWrapper
@Exirel Exirel changed the title Sopel 8.1: replace SopelWrapper by Sopel in the code, and add code deprecation warning to prevent its future usage Replace SopelWrapper by Sopel in the code, and add code deprecation warning to prevent its future usage Oct 21, 2024
@Exirel Exirel added this to the 8.1.0 milestone Oct 21, 2024
@Exirel Exirel added Breaking Change Stuff that probably should be mentioned in a migration guide Tests Documentation and removed Breaking Change Stuff that probably should be mentioned in a migration guide labels Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant