Skip to content

feat(django): upgrade django framework and deps#141

Merged
Cryptophobia merged 1 commit into
teamhephy:masterfrom
Cryptophobia:feature/upgrade-django-framework-and-deps
Mar 21, 2021
Merged

feat(django): upgrade django framework and deps#141
Cryptophobia merged 1 commit into
teamhephy:masterfrom
Cryptophobia:feature/upgrade-django-framework-and-deps

Conversation

@Cryptophobia
Copy link
Copy Markdown
Member

@Cryptophobia Cryptophobia commented Mar 20, 2021

Signed-off-by: Cryptophobia aouzounov@gmail.com

Copy link
Copy Markdown
Contributor

@yebyen yebyen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@Cryptophobia Cryptophobia force-pushed the feature/upgrade-django-framework-and-deps branch 2 times, most recently from cd67e6c to 5b3fb67 Compare March 21, 2021 05:20
  - upgrading django to 2.2.19 and djangorestframework to 3.12.2
  - resolving all failing tests due to above upgrade
  - upgrade models to default `on_delete=models.CASCADE` if app or user are deleted
  - upgrade all other dependencies to latest versions
  - update Makefile to use hephy/shell-dev for tests
  - update Dockerfile to use `hephy/base:latest` for building
  - update Dockefile to use python3 -m pip commands
  - upgrade pip, setuptools, wheel using python3 command
  - resolves teamhephy#140
  - resolves teamhephy#80

Signed-off-by: Cryptophobia <aouzounov@gmail.com>
@Cryptophobia Cryptophobia force-pushed the feature/upgrade-django-framework-and-deps branch from 5b3fb67 to fa49020 Compare March 21, 2021 05:32
@Cryptophobia Cryptophobia merged commit 843cb4f into teamhephy:master Mar 21, 2021
Copy link
Copy Markdown
Member

@kingdonb kingdonb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Address deprecation warnings and upgrade djangoRESTframework

3 participants