Skip to content
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

Make local testing use django 3. #242

Merged
merged 1 commit into from
Jun 25, 2024

Conversation

BBooijLiewes
Copy link
Contributor

Local testing uses a different package install process then the CI. To prevent us running postgres 12 default while local testing bu use 11 instead we must also use django 3.

To enable this we in the setup.py which is used for local testing require a django 3 version instead of a django 3 or 4 version. Since django 4 is only postgres 12+ compatible.

Local testing uses a different package install process then the CI. To
prevent us running postgres 12 default while local testing bu use 11
instead we must also use django 3.

To enable this we in the setup.py which is used for local testing
require a django 3 version instead of a django 3 or 4 version.
Since django 4 is only postgres 12+ compatible.
@BBooijLiewes BBooijLiewes requested a review from BlayeeR June 25, 2024 13:31
@BBooijLiewes BBooijLiewes self-assigned this Jun 25, 2024
Copy link
Contributor

@BlayeeR BlayeeR left a comment

Choose a reason for hiding this comment

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

LGTM, worth noting that github workflow is testing for both django 3 and django 4 respectively, so this change only affects local dev environment

@BlayeeR BlayeeR merged commit c58f0e4 into multiput-proxy-submodel-fix Jun 25, 2024
8 checks passed
BBooijLiewes added a commit that referenced this pull request Jun 25, 2024
Local testing uses a different package install process then the CI. To
prevent us running postgres 12 default while local testing bu use 11
instead we must also use django 3.

To enable this we in the setup.py which is used for local testing
require a django 3 version instead of a django 3 or 4 version.
Since django 4 is only postgres 12+ compatible.

Co-authored-by: Bob Booij-Liewes <[email protected]>
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.

2 participants