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

remove python_2_unicode_compatible that no longer exists in django 3 #95

Closed
wants to merge 2 commits into from

Conversation

msarahan
Copy link

@msarahan msarahan commented Feb 6, 2020

fixes #91

this arguably breaks python 2 in some ways. I can spend a bit more time on this if you'd prefer to maintain python 2 support, even though it is beyond its supported lifetime.

@LifeMoroz
Copy link

We have to provide information in README for compatible versions.
Can you do it here?

@msarahan
Copy link
Author

Yes, will do. Thanks!

@moritz89
Copy link

moritz89 commented Apr 5, 2020

Due to lack of activity, should I recreate a PR with an update to the README? This would probably result in a version bump from 0.2.1 to 0.3.0?

@msarahan
Copy link
Author

msarahan commented Apr 5, 2020

My apologies. I got sidetracked between a new job and all this COVID stuff. I will push a commit momentarily.

@banagale banagale self-assigned this Apr 10, 2020
@banagale banagale added this to the Version 0.2.2 Release milestone Apr 10, 2020
@banagale banagale changed the base branch from master to develop May 10, 2020 04:31
@banagale banagale self-requested a review May 10, 2020 04:31
@banagale
Copy link
Collaborator

I did not merge this because I wanted to make the commits individually. This was helpful, particularly the discussion on the version bump and compatibility.

In lieu of this pull, these commits handle the changes:

  • Remove deprecated python 2.7 support: 51a02df
  • Update setup.py, bumping version and updating package classifiers to updated requirements as described here.

Many thanks to @msarahan, @moritz89 and @LifeMoroz for contributing to this PR.

@banagale banagale closed this May 10, 2020
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.

Does django-address works with Django 3.0.2?
4 participants