chore: привести инфраструктуру к канону кампании и вынести приложение articles - #10
Merged
Conversation
Версия рантайма переезжает в .python-version и берётся оттуда uv-ом, поэтому setup-python из CI убран. В dependabot добавлены prefix chore и labels, добавлен линт заголовка PR. В Makefile check больше не гоняет линтер дважды, а линтер и тесты в CI разведены по шагам. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
django-admin startapp кладёт приложение рядом с пакетом проекта, а не внутрь него. Приложение переехало из python_django_blog/articles в articles, шаблоны из python_django_blog/templates/articles в articles/templates/articles. Метка приложения не изменилась: django берёт её из последнего компонента name, поэтому миграции остаются валидными, makemigrations --check это подтверждает. Заодно поправлены source в .coveragerc и цель test-coverage, которые считали покрытие только по пакету проекта. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Пачка python-репозиториев кампании
docs/components-modernization.md. Два коммита: инфраструктура и отдельно переезд приложения.Инфраструктура
.python-version(3.14, как в образах кита),setup-pythonиз CI убран: питон ставит uv по этому файлуrequires-pythonподнят до>=3.14labelsиcommit-message.prefix: choreastral-sh/setup-uv@v10.0.0вместоpip install uv, линтер и тесты разведены по шагамMakefile:checkбольше не гоняет линтер дваждыПереезд приложения
django-admin startappкладёт приложение рядом с пакетом проекта, а не внутрь него, поэтомуpython_django_blog/articles→articles, а шаблоныpython_django_blog/templates/articles→articles/templates/articles. Правки:apps.py(name),INSTALLED_APPS,include()в корневомurls.py, импорты внутри приложения,sourceв.coveragercи цельtest-coverage(они считали покрытие только по пакету проекта).Метка приложения при этом не меняется: django берёт её из последнего компонента
name, иpython_django_blog.articlesсarticlesдают одну и ту же меткуarticles. Существующие миграции остаются валидными, подтвержденоmanage.py makemigrations --check --dry-run→No changes detected.Радиус проверен: уроки ссылаются на этот репозиторий как на «репозиторий готового проекта» (
spec.ymlкурса python_django_basics), раскладку каталогов не показывают, поэтому правок в курсах переезд не требует.Проверено локально:
make lint,make test(10 тестов) зелёные на 3.14.7, образ собранdocker build— джобаdeployстоит подif: github.event_name == 'push'и на PR не проверяется.🤖 Generated with Claude Code