openedx / edx-django-utils

edX utilities for Django Application development.
https://edx.readthedocs.io/projects/edx-django-utils/en/latest/
Apache License 2.0
26 stars 20 forks source link

SE-3724: squash warnings #86

Closed eLRuLL closed 3 years ago

eLRuLL commented 3 years ago

Description:

This PR squashes the following warning (as part of the Squash a Warning event):

edx_django_utils/monitoring/tests/test_middleware.py::TestMonitoringMemoryMiddleware::test_memory_monitoring_when_enabled /home/elrull/workspace/opencraft/edx-django-utils/edx_django_utils/monitoring/internal/middleware.py:180: DeprecationWarning: memory_info_ex() is deprecated and will be removed; use memory_info() instead 'ext_memory_info': process.memory_info_ex(),

It can be checked running tests:

make test

Merge checklist:

Post merge:

openedx-webhooks commented 3 years ago

Thanks for the pull request, @eLRuLL! I've created OSPR-5301 to keep track of it in JIRA, where we prioritize reviews. Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket:

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

eLRuLL commented 3 years ago

@edx-webhook this is ready for review

openedx-webhooks commented 3 years ago

@eLRuLL 🎉 Your pull request was merged!

Please take a moment to answer a two question survey so we can improve your experience in the future.

natabene commented 3 years ago

@eLRuLL Thank you for your contribution.