-
Notifications
You must be signed in to change notification settings - Fork 15
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
updated the server/requirements.txt to latest version #88
base: development
Are you sure you want to change the base?
Conversation
Looks Ok. As far as I remember, there is no breakage when jumping from Django 4.2 to 5.0. |
Really? I'm surprised they bump the major if there's no breaking changes then, but let me check... |
Doesn't seem compatible, see "release cadence": https://docs.djangoproject.com/en/dev/internals/release-process/ |
Have these changes been tested somewhere @AviGawande? |
Not yet , because there was a django-package depreciation in it. |
Let's not merge untested dependencies upgrades, that's dangerous, specially with a major update on Django itself. Let us know when you test it out. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please test, there's too many breaking updates.
Ok I will test on my branch and let you know |
Till now there are lot of deprecated version of the server/requirements.txt of eventyay-video and due to this deprecations the cause an error while running the development environment locally. Prior to this PR the dependabot had tried to automate the updations but the changes are not really reflected in the codebase.