-
Notifications
You must be signed in to change notification settings - Fork 0
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
build: Replace django-storages master with pjsier:update-azure-storag… #81
Conversation
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.
Are the additional entries in requirements.txt the deps of django-storages[azure]?
Pipfile
Outdated
@@ -45,6 +43,8 @@ PyNaCl = "==1.4.0" | |||
PyYAML = "==5.3.1" | |||
psycopg2-binary = ">=2.9.1" | |||
django-apscheduler = ">=0.6.0" | |||
django-storages = {editable = true, ref = "update-azure-storage", git = "https://github.com/pjsier/django-storages.git"} |
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.
Why are you installing it as editable?
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.
Also, I think you should be able to specify a commit so that it doesn't get accidentally upgraded?
3eda22b
of azure-storage-blob 12.8.1, i think |
#81) * build: Replace django-storages master with pjsier:update-azure-storage branch * fix: Django-storages is editable
#Description
Installed django-storages's branch jschneier/django-storages#805 that supports
azure-storage-blob
versions >=12.0.0.Dependency changes
django-storages = {version = "==1.11.1", extras = ["azure"]}
withdjango-storages = {editable = true, ref = "update-azure-storage", git = "https://github.com/pjsier/django-storages.git"}
azure-storage-blob
from2.1.0
to12.8.1
Testing
No
Documentation
No
Migrations (if applicable)
Have any database migrations been committed as part of this pull request?
Checklist: