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

Django 3, 4 support fix #43

Merged
merged 29 commits into from
Feb 22, 2022
Merged

Django 3, 4 support fix #43

merged 29 commits into from
Feb 22, 2022

Conversation

temi4
Copy link
Contributor

@temi4 temi4 commented Feb 19, 2022

Сделал минимальные изменения:

  • убрал поддержку python2
  • добавил поддержку django 3 и 4
  • убрал поддержку django 1 и 2
  • обновил django-nose до nose2
  • все тесты поправил и проверил, все прошло и работает (на М1 нет рабочих python3.6, 3.7 на них проверить не смог)
  • сам код не причесывал (если нужно смогу сделать следующем PR)

.gitignore Outdated Show resolved Hide resolved
doc/examples-django/pytilsex/settings.py Outdated Show resolved Hide resolved
doc/examples/numeral.in_words.py Outdated Show resolved Hide resolved
pytils/templatetags/pytils_numeral.py Outdated Show resolved Hide resolved
pytils/templatetags/pytils_translit.py Outdated Show resolved Hide resolved
pytils/test/templatetags/helpers.py Outdated Show resolved Hide resolved
@last-partizan
Copy link
Owner

Если собираешься и дальше причёсывать код, это отлично, но было бы неплохо для начала настроить запуск тестов через github actions, и наверное заменить раннер тестов с nose на pytest. Это уже после этого PR, тут всё отлично, кроме тех комментариев что я отписал.

Changelog Outdated Show resolved Hide resolved
@temi4
Copy link
Contributor Author

temi4 commented Feb 20, 2022

Давай просто согласуем список задач/идей. Что для тебя важно, чтобы изменилось. Я какие-то свои предложения внесу. И в следующем PR их сделаю.

pytils/utils.py Outdated Show resolved Hide resolved
@last-partizan
Copy link
Owner

Github-actions я настроил, можешь смержиться с мастером и теперь тесты запустятся для всех версий что надо

@temi4
Copy link
Contributor Author

temi4 commented Feb 20, 2022

Github-actions я настроил, можешь смержиться с мастером и теперь тесты запустятся для всех версий что надо

Такие вопросы:

  1. tox.ini [gh-actions] - тут питон2, он по идее уже не нужен, мне править файл?
  2. test.yml мне выходит тоже подправить под свои задачи?

temi4 and others added 6 commits February 20, 2022 14:19
Co-authored-by: Serhii Tereshchenko <serg.partizan@gmail.com>
Co-authored-by: Serhii Tereshchenko <serg.partizan@gmail.com>
@temi4
Copy link
Contributor Author

temi4 commented Feb 20, 2022

Настройки для тестов пофиксил. Все работает гуд. Тесты прошли

This was referenced Feb 21, 2022
@last-partizan
Copy link
Owner

Зря, конечно, ты в этом же мерже поубирал префиксы "u", это только усложнило ревью.

Лучше бы это было сделать потом. Но ладно, всё вроде окей, мержу.

@last-partizan last-partizan merged commit 7c4ecad into last-partizan:master Feb 22, 2022
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