Skip to content

Убрать скрипт github-sync - #416

Merged
itrous merged 1 commit into
developfrom
chore/retire-github-sync
Oct 14, 2026
Merged

itrous merged 1 commit into
developfrom
chore/retire-github-sync

Conversation

@itrous

@itrous itrous commented Oct 13, 2026

Copy link
Copy Markdown
Owner

Что удалено

  • scripts/github-sync.sh и scripts/test-github-sync.sh. Скрипт публиковал релизный сквош поверх develop напрямую. Теперь develop на GitHub защищена, и изменения попадают в неё только через PR, поэтому синк больше не нужен.
  • В CLAUDE.md правило «push только в origin, не в github-зеркало» заменено одной строкой: origin — GitHub itrous/bsl-analyzer, develop защищена, изменения попадают в неё через PR.

Что оставлено и почему

  • scripts/github-mirror-exclude.txt — список путей, которые не должны попадать в публичное дерево. Это политика, и она остаётся без изменений.
  • xtask/tests/mirror_exclusions.rs — читает только этот список и от скрипта не зависит. Тест не даёт исходникам встраивать при компиляции файлы из исключённых путей.

Открытые вопросы до слияния

  • Комментарии в scripts/install.ps1 (строка 33) и в scripts/github-mirror-exclude.txt (строка 2) всё ещё ссылаются на удалённый скрипт.
  • Значения INSTALL_SOURCE в scripts/install.sh и scripts/install.ps1, маркеры INSTALL_URL в README.md и crates/bsl-launcher/release-source.json уже содержат github. Раньше синк подставлял их при публикации, теперь это нужно поддерживать вручную.

Как публиковать изменения теперь

Изменения вносятся веткой и открываются PR в develop. Прямой push в защищённую develop не используется, отдельного синк-скрипта больше нет.

Проверка

  • cargo test -p xtask: mirror_exclusions, 2 из 2 тестов прошли.
  • Pre-commit хук (fmt, clippy, cargo test --all) в свежем клоне не установлен, полный прогон не запускался. Изменений Rust-кода в PR нет.

Удалены scripts/github-sync.sh и scripts/test-github-sync.sh: develop на GitHub
теперь защищена, и синк, который клал релизный сквош поверх неё в обход PR,
больше не нужен. Список исключений github-mirror-exclude.txt и тест
mirror_exclusions остаются: это политика, а не часть скрипта.
@itrous
itrous merged commit f63a2eb into develop Oct 14, 2026
4 checks passed
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