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

fix: django.contrib.postgres.fields.JSONField causes an error in Django 4.2 (fields.E904) https://docs.djangoproject.com/en/4.2/ref/checks/ #154

Merged
merged 6 commits into from
Nov 11, 2024

Conversation

jpmckinney
Copy link
Member

No description provided.

@jpmckinney jpmckinney requested review from hancush and fgregg November 11, 2024 20:34
Copy link
Contributor

@fgregg fgregg left a comment

Choose a reason for hiding this comment

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

@jpmckinney would you mind fixing the tests?

For example:

env \
  GEOS_LIBRARY_PATH='/opt/homebrew/opt/geos/lib/libgeos_c.dylib' \
  GDAL_LIBRARY_PATH='/opt/homebrew/opt/gdal/lib/libgdal.dylib' \
  DATABASE_DB=opencivicdata_test \
  DATABASE_USER=myuser \
  DATABASE_PASSWORD= \
  PYTHONPATH=. pytest --ds=opencivicdata.tests.test_settings
@coveralls
Copy link

coveralls commented Nov 11, 2024

Coverage Status

coverage: 98.898% (+0.001%) from 98.897%
when pulling 80c189f on django42
into d5520de on master.

@jpmckinney
Copy link
Member Author

@fgregg All green now.

@fgregg
Copy link
Contributor

fgregg commented Nov 11, 2024

thanks for this, @jpmckinney. hope all is well with you

@fgregg fgregg merged commit d1114e1 into master Nov 11, 2024
13 checks passed
@fgregg fgregg deleted the django42 branch November 11, 2024 21:01
@jpmckinney
Copy link
Member Author

All good here :) It looks like the commit to roll the version to 3.3.1 is not on master. It was also tagged a "v3.3.1" (like v3.3.0). However, the other tags omit the "v".

Okay for me to tag and release as "3.4.0" ?

@fgregg
Copy link
Contributor

fgregg commented Nov 11, 2024

go for it!

@jpmckinney
Copy link
Member Author

Ah, I see why the "v" was included - the workflow only runs on "v*" tags. I'll use it.

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

Successfully merging this pull request may close these issues.

3 participants