Skip to content

DL-2026: имя джобы test вместо имени workflow Lab Tests - #61

Merged
markpolyak merged 3 commits into
mainfrom
fix/dl-2026-ci-jobs
Oct 27, 2026
Merged

markpolyak merged 3 commits into
mainfrom
fix/dl-2026-ci-jobs

Conversation

@markpolyak

Copy link
Copy Markdown
Owner

Проблема

Проверка ЛР1 отвечала:

⚠️ Обязательные джобы не найдены среди проверок CI: Lab Tests.

В ci.workflows перечисляются имена check runs, то есть имена джоб. Имя workflow туда не попадает — оно живёт только в UI Actions. В tests.yaml курса:

name: Lab Tests      # ← имя workflow, стояло в конфиге
jobs:
  test:              # ← имя джобы, с ним и приходит check run

На itmo-dl-2026/1-markpolyak (коммит 987e041d) единственный check run называется test и завершился успешно — совпадения с Lab Tests не было, поэтому проверка честно отказывалась выставлять оценку.

Что сделано

  • ЛР1: ci.workflows → test.
  • Закомментированные ЛР2–ЛР6 переведены со старого формата ci: [- workflows] на тот же явный список: при расскомментировании они оценивались бы по списку имён по умолчанию, то есть по чему угодно, что окажется в репозитории.
  • Контрольная (quiz) не тронута: у неё отдельный шаблон, которого ещё нет, ссылка открывается 31.12.2026.

Проверка

С новым конфигом 1-markpolyak получает v (1/1) на тех же реальных check runs.

Альтернатива — дать джобе name: Lab Tests в общем шаблоне course-maker и раскатать на репозитории курса; отклонена как более дорогая.

🤖 Generated with Claude Code

markpolyak and others added 3 commits September 25, 2026 05:13
В ci.workflows перечисляются имена check runs, то есть имена джоб, а имя
workflow туда не попадает. Конфиг требовал "Lab Tests" - это `name:` самого
workflow в tests.yaml, - тогда как джоба в нём называется `test` и check run
приходит именно с этим именем. Отсюда CI_JOBS_NOT_FOUND у 1-markpolyak,
хотя тесты пройдены.

Закомментированные ЛР2-ЛР6 заодно переведены со старого формата
`ci: [- workflows]`, который конкретные джобы не называет: при
расскомментировании они оценивались бы по списку имён по умолчанию.

Проверено на itmo-dl-2026/1-markpolyak: единственный check run называется
`test` и завершился успешно, с новым конфигом лаба получает "v" (1/1).

Контрольная (quiz) не тронута: у неё отдельный шаблон, которого ещё нет.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@markpolyak
markpolyak merged commit f90780b into main Oct 27, 2026
3 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