Skip to content

docs: исправить ложное «у deploy.yml нет paths-фильтра» - #11

Merged
bronxtc52 merged 1 commit into
mainfrom
docs/fix-paths-filter-claim
Jul 18, 2026
Merged

docs: исправить ложное «у deploy.yml нет paths-фильтра»#11
bronxtc52 merged 1 commit into
mainfrom
docs/fix-paths-filter-claim

Conversation

@bronxtc52

Copy link
Copy Markdown
Owner

Ошибка, которую исправляю

В #8 и #10 (оба уже в main) я написал «у deploy.yml нет paths-фильтра, любой мерж передеплоит бота». Это неверно.

Фильтр есть:

paths: ["bot/**", ".github/workflows/deploy.yml"]

Проверено на живых мерджах этой сессии: #8 и #10 тронули только корневой CLAUDE.md → запустился лишь tests, деплой не стартовал, ревизия осталась sha-a22d7a6.

Корень моей ошибки

После мерджа #7 я увидел смену ревизии и заключил «доки деплоят → фильтра нет» — не прочитав сам workflow. Но #7 тронул bot/ops/provision-prod.sh, а это под bot/**. 409-конфликты в 12:03–04 были от #9 (код), не от доков. Я поймал это, проверив факт вместо доверия памяти.

Что остаётся верным

Грабля-наоборот: bot/ops/* выглядит «оперативной» правкой, но это bot/**деплоит. И 409 на пересменке при реальном деплое — ожидаемая пересменка.

Доки-онли. Этот мерж, по свежеустановленному факту, бота не передеплоит 🙂

🤖 Generated with Claude Code

Фильтр ЕСТЬ: paths: ["bot/**", ".github/workflows/deploy.yml"]. Мерж правок
только в корневой CLAUDE.md передеплой НЕ запускает — идёт лишь tests.
Проверено на живых мерджах #8 и #10 (только CLAUDE.md → deploy не стартовал).

Корень ошибки: после мерджа #7 я увидел смену ревизии и заключил «доки
деплоят → фильтра нет», не прочитав сам workflow. Но #7 тронул
bot/ops/provision-prod.sh — это под bot/**, поэтому и деплой. 409 в 12:03-04
были от #9 (код), не от доков.

Грабля-наоборот верна и полезна: bot/ops/* выглядит «оперативным», но это
bot/** → деплоит.

Co-Authored-By: Claude <noreply@anthropic.com>
@bronxtc52
bronxtc52 merged commit 1986c34 into main Jul 18, 2026
1 check passed
@bronxtc52
bronxtc52 deleted the docs/fix-paths-filter-claim branch July 18, 2026 03:12
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