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

Cleanup #97

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Cleanup #97

wants to merge 8 commits into from

Conversation

Djailla
Copy link

@Djailla Djailla commented May 18, 2020

  • Remove Python 2.7 support
  • Fix Django deprecation warning

Bastien Vallet added 4 commits May 15, 2020 17:14
url_filter/validators.py:22: RemovedInDjango40Warning: django.utils.translation.ungettext_lazy() is deprecated in favor of django.utils.translation.ngettext_lazy().
UTF-8 is default in Python3
@Djailla Djailla force-pushed the cleanup branch 6 times, most recently from 154a7ac to d88d62c Compare May 18, 2020 10:36
- Use pycqa flake8 version
- Remove default --safe black option
@Djailla
Copy link
Author

Djailla commented May 18, 2020

Hello @miki725

I'm providing a review to cleanup django deprecation warning and trying to remove six dependency, but struggle understanding why CI fails for codecov, could you help me as I'm not an expert with Drone ci ?

thanks

@Djailla
Copy link
Author

Djailla commented Jun 4, 2020

Hello @miki725 how are you ? Any update on this PR ?

@Djailla
Copy link
Author

Djailla commented Nov 22, 2020

No news for few month :(

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.

1 participant