fix(ci): move ci.yml to .forgejo/workflows to trigger pull_request #12

Merged

slaid098 merged 1 commit from fix/ci-move-to-forgejo-workflows into main

2026-08-07 17:41:40 +03:00

slaid098 commented

2026-08-07 17:38:36 +03:00

Owner

Что сделано

Перенос ci.yml из .github/workflows/ в .forgejo/workflows/ (рядом с deploy.yml). Удалён пустой .github/ каталог.

Изменения

git mv .github/workflows/ci.yml .forgejo/workflows/ci.yml (rename, контент не менялся)
rmdir .github/workflows && rmdir .github (каталоги пустые после переноса)

Почему

После переноса deploy.yml в .forgejo/workflows/ (PR #6) Forgejo перестал сканировать .github/workflows/. Доказательство: POST /actions/workflows/ci.yml/dispatches → 404 «workflow not found». PR #9 и PR #11 — ci.yml не запустился на pull_request (нет CI checks в pipeline-status). deploy.yml в .forgejo/ работает (runs 106-124).

Forgejo поддерживает оба каталога (.github/workflows/ и .forgejo/workflows/) как legacy-совместимость, но при наличии .forgejo/ — .github/ перестаёт триггериться (поведение подтверждено на Forgejo 16.0.2).

Watch out

Chicken-and-egg: этот PR меняет workflow-файл. Forgejo может не запустить ci.yml на этом PR (новый workflow-файл требует explicit enable в UI Actions → Enable). Если CI не запустится — мержить по review APPROVE + локальным проверкам. После merge push-триггер проверит ci.yml на main.
Контент ci.yml не менялся — runs-on: ubuntu-latest, node-version: 22, cache: npm, steps: lint → typecheck → knip → test → build. Biome check на ветке — 0 ошибок (rename без правок контента).
.github/ удалён полностью — других файлов в нём не было (dependabot.yml удалён в PR #6, workflows/ci.yml перенесён).
pipeline-status — после merge этого PR перестанет возвращать AMBIGUOUS на новых PR (CI checks будут presence). Workaround «мерж без CI» больше не нужен.

Pending

Проверить после merge: push на main триггерит ci.yml (run с event=push, status=success/failure). Если нет — workflow требует активации в web UI Forgejo (Actions → выбрать ci.yml → Enable).
Обновить run-pipeline skill: Template D «деталей через API нет» — устарел для Forgejo 16.0.2+ (логи доступны через API /actions/jobs/{id}/logs).

Closes #10

## Что сделано Перенос `ci.yml` из `.github/workflows/` в `.forgejo/workflows/` (рядом с `deploy.yml`). Удалён пустой `.github/` каталог. ### Изменения - `git mv .github/workflows/ci.yml .forgejo/workflows/ci.yml` (rename, контент не менялся) - `rmdir .github/workflows && rmdir .github` (каталоги пустые после переноса) ## Почему После переноса `deploy.yml` в `.forgejo/workflows/` (PR #6) Forgejo перестал сканировать `.github/workflows/`. Доказательство: `POST /actions/workflows/ci.yml/dispatches` → 404 «workflow not found». PR #9 и PR #11 — ci.yml не запустился на `pull_request` (нет CI checks в pipeline-status). `deploy.yml` в `.forgejo/` работает (runs 106-124). Forgejo поддерживает оба каталога (`.github/workflows/` и `.forgejo/workflows/`) как legacy-совместимость, но при наличии `.forgejo/` — `.github/` перестаёт триггериться (поведение подтверждено на Forgejo 16.0.2). ## Watch out - **Chicken-and-egg**: этот PR меняет workflow-файл. Forgejo может не запустить ci.yml на этом PR (новый workflow-файл требует explicit enable в UI Actions → Enable). Если CI не запустится — мержить по review APPROVE + локальным проверкам. После merge push-триггер проверит ci.yml на main. - **Контент ci.yml не менялся** — `runs-on: ubuntu-latest`, `node-version: 22`, `cache: npm`, steps: lint → typecheck → knip → test → build. Biome check на ветке — 0 ошибок (rename без правок контента). - **`.github/` удалён полностью** — других файлов в нём не было (dependabot.yml удалён в PR #6, workflows/ci.yml перенесён). - **`pipeline-status`** — после merge этого PR перестанет возвращать AMBIGUOUS на новых PR (CI checks будут presence). Workaround «мерж без CI» больше не нужен. ## Pending - Проверить после merge: push на main триггерит ci.yml (run с event=push, status=success/failure). Если нет — workflow требует активации в web UI Forgejo (Actions → выбрать ci.yml → Enable). - Обновить `run-pipeline` skill: Template D «деталей через API нет» — устарел для Forgejo 16.0.2+ (логи доступны через API `/actions/jobs/{id}/logs`). Closes #10

slaid098 added 1 commit

2026-08-07 17:38:36 +03:00

fix(ci): move ci.yml to .forgejo/workflows to trigger pull_request

CI / check (pull_request) Successful in 1m57s

Details

53ae66b8d2

- после переноса deploy.yml в .forgejo/workflows (PR #6) Forgejo перестал сканировать .github/workflows/
- ci.yml не триггерился на pull_request (PR #9, #11 — нет CI checks), dispatch → 404
- перенос ci.yml в .forgejo/workflows/ + удаление пустого .github/
- контент ci.yml не менялся (lint+typecheck+knip+test+build)

Closes #10

slaid098 commented

2026-08-07 17:41:35 +03:00

Author

Owner

Code Review Summary

Чистый rename ci.yml из .github/workflows/ в .forgejo/workflows/ — восстанавливает триггер pull_request для Forgejo Actions (issue #10).

Проверки

Чистый rename: git diff main...HEAD → similarity index 100%, 0 insertions, 0 deletений. Контент не тронут.
ci.yml контент: триггеры pull_request + push на main, jobs: checkout → setup-node → npm ci → lint → typecheck → knip → test → build — идентичен оригиналу.
.github/ удалён: каталога нет (ls .github → not found), пустого каталога не осталось.
.forgejo/workflows/: содержит ci.yml + deploy.yml — структура консистентна.
Forgejo совместимость: actions/checkout@v4, actions/setup-node@v4 — те же actions, что в deploy.yml, поддерживаются Forgejo.
CI green: run id=127, event=pull_request — триггер работает, фикс подтверждён.
Cross-file impact: rg по репо не нашёл ссылок на .github/workflows/ci.yml или .forgejo/workflows в коде/документации — readers отсутствуют, breakage risk = 0.

Positives

Минимальный, точечный фикс — ровно то, что нужно для issue #10.
Использован git mv → сохранена история файла (rename detection).
Удалён пустой .github/ каталог — нет мусора.

Готов к merge.

Verdict: APPROVE

## Code Review Summary Чистый rename `ci.yml` из `.github/workflows/` в `.forgejo/workflows/` — восстанавливает триггер `pull_request` для Forgejo Actions (issue #10). ### Проверки - **Чистый rename**: `git diff main...HEAD` → `similarity index 100%`, 0 insertions, 0 deletений. Контент не тронут. - **ci.yml контент**: триггеры `pull_request` + `push` на `main`, jobs: checkout → setup-node → npm ci → lint → typecheck → knip → test → build — идентичен оригиналу. - **`.github/` удалён**: каталога нет (`ls .github` → not found), пустого каталога не осталось. - **`.forgejo/workflows/`**: содержит `ci.yml` + `deploy.yml` — структура консистентна. - **Forgejo совместимость**: `actions/checkout@v4`, `actions/setup-node@v4` — те же actions, что в `deploy.yml`, поддерживаются Forgejo. - **CI green**: run id=127, event=pull_request — триггер работает, фикс подтверждён. - **Cross-file impact**: `rg` по репо не нашёл ссылок на `.github/workflows/ci.yml` или `.forgejo/workflows` в коде/документации — readers отсутствуют, breakage risk = 0. ### Positives - Минимальный, точечный фикс — ровно то, что нужно для issue #10. - Использован `git mv` → сохранена история файла (rename detection). - Удалён пустой `.github/` каталог — нет мусора. Готов к merge. ### Verdict: APPROVE

slaid098 merged commit fafa5889a3 into main

2026-08-07 17:41:40 +03:00

slaid098 deleted branch fix/ci-move-to-forgejo-workflows

2026-08-07 17:41:40 +03:00

slaid098 referenced this pull request from a commit

2026-08-07 17:41:41 +03:00

fix(ci): move ci.yml to .forgejo/workflows to trigger pull_request (#12)

slaid098 referenced this pull request

2026-08-07 17:42:47 +03:00

fix(ci): remove npm cache from setup-node to fix push trigger failure #13