Upgrade dependencies #318
Merged
Upgrade dependencies #318
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Now that we use docker, this script isn't needed for deployment
Remove explicit Werkzeug and click dependencies as these are added by Flask
|
The new version of click has changed functions of the form |
From click 7, _ in function names are converted to - (e.g. init_db -> init-db). In order to keep our configuration and documentation up-to-date, specify command names manually until we decide which format to use.
|
@paramsingh this is working again, I updated |
|
Thanks, merged! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR upgrades some dependencies indicated by github as having security updates, and also does a general python version upgrade (from
pip list --outdated). I left some outdated packages at their normal version because these are dependencies of brainzutils, and should be updated there.Some js dependencies of gulp have issues that can only be fixed by upgrading to gulp4. Doing this as a second step as it will probably require changes to the gulpfile too.
upgrades to python packages tested manually: flask-admin (go to admin), click (run commands), ndg-httpsclient (perform oauth login), pytest (run tests)