GitHub CI пропускает черновик релиза при разрешённом пропуске Windows-подписи

Open Beginner friendly
  • Is something wrong?
#336 0 comments 0 reactions 0 assignees View on GitHub

Maintainers usually reply within 1 day

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
78/100
Issue type
Bug
Clarity
Clearly specified
Activity status
Active
Tech stack
csharp, github-actions
Domain
ci-cd

Research direction

Start with the package and release jobs in .github/workflows/ci.yml around lines 459–469 and 523–527, then review the linked GitHub Actions dependency behavior. Make the release job run for release tags on GitHub after successful packaging, including the permitted unsigned case. Check signing success, permitted signing skip, and package failure.

Written by the indexing model from the issue text.

Description

bug github_actions

Приоритет P2 · milestone v0.5 (0.5.0)

При выпуске с явно разрешённым исключением для Windows-подписи CI собирает и проверяет архивы, но молча пропускает создание черновика.

Воспроизведение: открыть прогон v0.4.0. Sign Windows binaries имеет статус skipped, Verify release archives — success, Draft release — skipped. Для этого выпуска черновик пришлось создавать отдельно из проверенного артефакта.

Причина: условие package в .github/workflows/ci.yml:459–469 явно допускает skipped для разрешённых тегов. У release (.github/workflows/ci.yml:523–527) нет явной функции статуса, поэтому GitHub применяет неявный success(), и пропуск транзитивной зависимости распространяется на финальное задание. Условия и зависимости GitHub Actions.

Ожидаемо: черновик создаётся на GitHub для релизного тега после успешного package, в том числе для разрешённого unsigned-исключения.

Направление исправления: проверять !cancelled() и needs.package.result == 'success' явно. Сохранить ограничение на релизный тег и GitHub. Проверить ветви «подписание выполнено», «разрешённый пропуск» и «упаковка упала».

Аудит ffa65648867139f9e45c15f4b635a83c1ff52c1b.

Dominant language
C#
Stars
5
Forks
1
Avg merge
23m
Merged PRs (30d)
144

Getting set up

This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from endercodezz/BYDLO

All issues in endercodezz/BYDLO

Similar issues

More C# issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.