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.

Sign up for GitHub

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

Jump to bottom

[Deprecation Warning]: The 'warn' method is deprecated, use 'warning' instead #149

Closed
UsamaSadiq opened this issue Jun 24, 2022 · 6 comments · Fixed by openedx/edx-platform#34057
Closed

[Deprecation Warning]: The 'warn' method is deprecated, use 'warning' instead #149

UsamaSadiq opened this issue Jun 24, 2022 · 6 comments · Fixed by openedx/edx-platform#34057
Assignees
@lewisemm
Labels
hacktoberfest Recommended issue for Hacktoberfest participants help wanted Ready to be picked up by anyone in the community maintenance Routine upkeep necessary for the health of the platform

Comments

@UsamaSadiq
Copy link
Member

Description

  • Python test warnings Suppression process has been started to squash/fix the existing pytest warnings from edx-platform.
  • The details to handle the warnings and current warnings report can be found in the document
@UsamaSadiq
Copy link
Member Author

label: help wanted

@UsamaSadiq
Copy link
Member Author

label: maintenance

@github-actions github-actions bot added help wanted Ready to be picked up by anyone in the community maintenance Routine upkeep necessary for the health of the platform labels Jun 28, 2022
@jmbowman jmbowman added the hacktoberfest Recommended issue for Hacktoberfest participants label Oct 16, 2022
@Younggregs
Copy link

Hello @UsamaSadiq is this issue still open?

@UsamaSadiq
Copy link
Member Author

@Younggregs the deprecation warnings occurring due to third party packages are still open.
You can see the latest warnings report from edx-platform github actions summary tab and see which warnings are still occurring.

@lewisemm
Copy link

assign me

lewisemm added a commit to lewisemm/edx-platform that referenced this issue Jan 15, 2024
@lewisemm
fix: Replace deprecated .warn method with .warning
26bcbb3 
The `logging.Logger.warn` method has been deprecated since Python 3.3
and is due to be removed all together in Python 3.13. See
python/cpython#105377

`logging.Logger.warning` is the preferred and recommended way to log
warnings.

Fixes openedx/public-engineering#149
@lewisemm lewisemm mentioned this issue Jan 15, 2024
Merged
@UsamaSadiq
Copy link
Member Author

@lewisemm Thanks for the contribution. I'll review your PR once the tests pass on it.

UsamaSadiq pushed a commit to openedx/edx-platform that referenced this issue Jan 16, 2024
@lewisemm
fix: Replace deprecated .warn method with .warning (#34057)
780e908 
The `logging.Logger.warn` method has been deprecated since Python 3.3
and is due to be removed all together in Python 3.13. See
python/cpython#105377

`logging.Logger.warning` is the preferred and recommended way to log
warnings.

Fixes openedx/public-engineering#149

Co-authored-by: Lewis M. Kabui <lewisemm@users.noreply.github.com>
mariajgrimaldi pushed a commit to eduNEXT/edunext-platform that referenced this issue Jan 23, 2024
@lewisemm @mariajgrimaldi
fix: Replace deprecated .warn method with .warning (#34057)
408e104 
The `logging.Logger.warn` method has been deprecated since Python 3.3
and is due to be removed all together in Python 3.13. See
python/cpython#105377

`logging.Logger.warning` is the preferred and recommended way to log
warnings.

Fixes openedx/public-engineering#149

Co-authored-by: Lewis M. Kabui <lewisemm@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lewisemm lewisemm

Labels
hacktoberfest Recommended issue for Hacktoberfest participants help wanted Ready to be picked up by anyone in the community maintenance Routine upkeep necessary for the health of the platform
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: Replace deprecated .warn method with .warning lewisemm/edx-platform
4 participants
@jmbowman @lewisemm @Younggregs @UsamaSadiq