Skip to content

Собранные PDF — в релизы, не в git - #7

Merged
yknnv merged 1 commit into
mainfrom
chore/dist-v-relizy
Sep 1, 2026
Merged

yknnv merged 1 commit into
mainfrom
chore/dist-v-relizy

Conversation

@yknnv

@yknnv yknnv commented Sep 1, 2026

Copy link
Copy Markdown
Owner

Что меняется

То же решение, что в sail-logbook: dist/ в .gitignore и снят
с отслеживания, готовые PDF живут в релизах.

Причина — не чистота ради чистоты. PDF пересобирается целиком при любой
правке текста, git такие блобы не дельтит, и каждая сборка оседала бы
в истории отдельными 12 МБ. Ссылка releases/latest при этом постоянная,
а release.yml уже собирает и прикладывает файлы по тегу.

Заодно src/logbook.py создаёт dist/ сам — иначе в свежем клоне сборка
падает на записи первого PDF. Ровно на это наткнулся CI в sail-logbook.

Проверено

  • rm -rf dist && ./build.sh с нуля: 130 страниц, 42 погружения
  • check_margins.py: нарушений нет, наложений нет
  • вёрстка не затронута, объём не изменился
  • README.md и README.en.md больше не отправляют читателя в dist/

Чего этот PR не делает

Историю не переписывает: старые блобы остаются в прошлых коммитах,
.git от них не похудеет. Это осознанно — переписывание сломало бы
тег v1.0 и потребовало force-push, который теперь запрещён правилом.
Смысл правки в том, чтобы репозиторий перестал расти дальше.

dist/ уходит в .gitignore и снимается с отслеживания. PDF пересобирается
целиком при каждой правке, git его не сжимает и не дельтит — так в историю
навсегда добавлялось бы по 12 МБ за сборку. Готовые файлы и так публикует
release.yml по тегу.

Каталог dist/ теперь создаётся кодом: в свежем клоне его нет, и reportlab
падал бы на записи первого же PDF.

README на обоих языках больше не зовёт читателя в dist/ за файлами.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@yknnv
yknnv merged commit bb3f80d into main Sep 1, 2026
1 check passed
@yknnv
yknnv deleted the chore/dist-v-relizy branch September 1, 2026 12:27
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