Skip to content

Исправление check.yml: прогон в CI даже при FAILED завершался успешно. - #180

Merged
petrCher merged 2 commits into
mainfrom
fix/fix-ci-checks-report
Sep 4, 2026
Merged

Исправление check.yml: прогон в CI даже при FAILED завершался успешно.#180
petrCher merged 2 commits into
mainfrom
fix/fix-ci-checks-report

Conversation

@CaseAsLimbo

@CaseAsLimbo CaseAsLimbo commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Изменения

  • Добавлена переменная id: pytest (в её отсутствие в steps.pytest.outcome хранилось null и получалось сравнение null == 'failure')
  • Добавлена инструкция для корректного выбора кода ошибки в pipe-ах в отсутствие --pipefail в оболочке
  • Исправлен запуск шага Fail on pytest error (никогда не запускался), а так же включен if: always() для шага Pytest coverage comment

Check-List

  • Вы написали тесты к реализованным функциям?
  • Вы проверили свой код перед отправкой запроса?
  • Вы не забыли применить форматирование black и isort для Back-End или Prettier для Front-End?

@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown

Code Coverage

Coverage Report
FileStmtsMissCoverMissing
rating_api
   __main__.py440%1–6
   exceptions.py43784%35–37, 48–50, 58
rating_api/models
   base.py64494%24–27
   db.py1621889%114, 116, 118, 120, 131, 170, 188, 198, 212, 221–223, 257, 271–283
rating_api/routes
   base.py16194%40
   comment.py1512782%76, 94, 191, 264–265, 267–268, 276–281, 290, 297–299, 329, 352, 392–403, 430
   exc_handlers.py32391%36, 43, 50
   lecturer.py109992%71–72, 82, 88–89, 229, 237, 259, 265
rating_api/schemas
   base.py12467%6–9
   models.py155398%189, 191, 201
rating_api/utils
   mark.py880%1–19
TOTAL7908889% 

Summary

Tests Skipped Failures Errors Time
94 0 💤 0 ❌ 0 🔥 19.686s ⏱️

@CaseAsLimbo
CaseAsLimbo force-pushed the fix/fix-ci-checks-report branch 2 times, most recently from e75a8db to 536328b Compare September 2, 2026 14:16
@CaseAsLimbo CaseAsLimbo changed the title Проверка вклюен ли параметр оболочки pipefail Исправление check.yml: прогон в CI даже при FAILED завершался успешно. Sep 2, 2026
@CaseAsLimbo
CaseAsLimbo force-pushed the fix/fix-ci-checks-report branch from 536328b to 3cd11dc Compare September 2, 2026 14:29
@CaseAsLimbo
CaseAsLimbo requested a review from petrCher September 2, 2026 14:31
@CaseAsLimbo
CaseAsLimbo force-pushed the fix/fix-ci-checks-report branch from 0de0d80 to 6ab7157 Compare September 2, 2026 15:10
Comment thread .github/workflows/checks.yml
Comment thread .github/workflows/checks.yml
@petrCher

petrCher commented Sep 4, 2026

Copy link
Copy Markdown
Member

не забывай делать assignees себя на свой пр))

@CaseAsLimbo
CaseAsLimbo requested a review from petrCher September 4, 2026 10:12
Comment thread .github/workflows/checks.yml
@petrCher
petrCher merged commit 28adaab into main Sep 4, 2026
2 checks passed
@petrCher
petrCher deleted the fix/fix-ci-checks-report branch September 4, 2026 18:07
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.

2 participants