Conversation
bmtcril
force-pushed
the
bmtcril/upgrade_django52
branch
from
dd97627 to
94a1d53
Compare
|
|
||
| __version__ = '0.5.1' | ||
|
|
||
| default_app_config = 'edx_event_bus_redis.apps.EdxEventBusRedisConfig' # pylint: disable=invalid-name |
There was a problem hiding this comment.
| include README.rst | ||
| include requirements/base.in | ||
| include requirements/constraints.txt | ||
| recursive-include edx_event_bus_redis *.html *.png *.gif *.js *.css *.jpg *.jpeg *.svg |
There was a problem hiding this comment.
This package doesn't actually include any UI, so warnings were being thrown at build time about the lack of these files.
| @@ -1,22 +0,0 @@ | |||
|
|
|||
There was a problem hiding this comment.
This file was never used, it just came as part of the cookiecutter.
| migrations | ||
|
|
||
| [wheel] | ||
| universal = 1 |
There was a problem hiding this comment.
Universal wheels were for Python 2.7 & 3 compatibility, they are no longer needed and warnings were being thrown at build time.
| install_requires=load_requirements("requirements/base.in"), | ||
| python_requires=">=3.11", | ||
| license="AGPL 3.0", | ||
| license="AGPL-3.0-or-later", |
There was a problem hiding this comment.
This more closely matches our removed classifier. https://packaging.python.org/en/latest/guides/writing-pyproject-toml/#license
| 'Framework :: Django', | ||
| 'Framework :: Django :: 4.2', | ||
| 'Intended Audience :: Developers', | ||
| 'License :: OSI Approved :: GNU Affero General Public License v3 or later (AGPLv3+)', |
There was a problem hiding this comment.
Trove classifiers for license are deprecated and this threw a warning.
| "context_processors": [ | ||
| "django.contrib.auth.context_processors.auth", # this is required for admin | ||
| "django.contrib.messages.context_processors.messages", # this is required for admin | ||
| "django.template.context_processors.request", # this is required for admin |
There was a problem hiding this comment.
This line is the only real change, it was throwing a warning at test time.
2 participants
Closes #163
Ran django-upgrade, which made one change in
__init__.py, see PR comment.Merge checklist: