Skip to content

check-coding-rule: 判定と reviewdog 投稿を分離する #134

Description

背景

arkedge/aocs-all#3758 では、c2a-anzu の coding-rule 判定を required CI に残しつつ、reviewdog による PR 書き込みを non-blocking な経路へ分離する。

現在の reusable workflow は判定と reviewdog の副作用が一体になっているため、required job に write permission が必要になる。

対応内容

  • coding-rule の検査と終了 status だけを返す read-only な呼び出し方を追加する
  • reviewdog の投稿を独立した job、workflow、または明示的に無効化できる mode へ分離する
  • read-only 実行では PR write permission と reviewdog token を要求しない
  • 判定失敗と reviewdog 投稿失敗を区別できるテストを追加する

完了条件

  • aocs-all の required CI が coding-rule 判定だけを contents: read で実行できる
  • reviewdog の障害や権限不足が merge を止めない
  • non-blocking な reviewdog 投稿機能は維持される

関連

  • arkedge/aocs-all#3758

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions