Skip to content

fix: Tracking一括更新でレビュー待ちを維持する - #7

Merged
shunsuke-shimomura merged 1 commit into
mainfrom
fix/preserve-needs-review
Sep 11, 2026
Merged

shunsuke-shimomura merged 1 commit into
mainfrom
fix/preserve-needs-review

Conversation

@shunsuke-shimomura

Copy link
Copy Markdown
Member

Description

MTG後のTracking一括更新で、議論中のアイテムを自動的に確認済みとして扱わないようにします。実装に残っていたNeeds Discussionの自動更新条件を削除し、Trackedへの自動更新をTracking未設定のアイテムだけに限定しました。

  • StatusがTodoIn Progressなどの場合、Needs Reviewは維持し、議論後に手動でTrackedへ変更します。
  • StatusがDoneならClosedIceboxならIceboxへの自動更新を優先します。TrackingがNeeds Reviewでも同様です。
  • READMEに残っていたNeeds ReviewからTrackedへの自動更新の説明を修正し、Iceboxの例外も明記しました。

Test results

  • prek run --files .github/workflows/update-tracking-status.yml README.md: 全チェック通過
  • actionlint .github/workflows/update-tracking-status.yml: 通過
  • yamllint --strict -c .yamllint.yaml .github/workflows/update-tracking-status.yml: 通過
  • ワークフローのrunスクリプトを模擬gh APIで実行。Status 4種類 × Tracking 6種類(未設定・Needs Review・Needs Discussion・Tracked・Closed・Icebox)の24通りで更新結果を検証。通常実行/dry-run、既定/カスタムフィールド名の4条件すべて通過。実Projectの更新は実施していません。
  • git diff --check: 通過
  • ローカルのzizmor 1.29.0ではGitHub Appトークン権限について既存の2件を検出。origin/mainでも同じ指摘が出ることを確認済みです。

Impact

update-tracking-status.ymlを呼び出すリポジトリに適用されます。Needs Discussionも通常の一括更新では維持されるため、必要に応じて手動でNeeds ReviewまたはTrackedへ整理してください。Done/Iceboxへの遷移、未設定アイテムの確認済み更新は維持します。

Supplementary information

Renovate専用のTracking更新ワークフローは変更対象に含みません。

@github-actions github-actions Bot added documentation 📓 Improvements or additions to documentation ci ⚙️ Changes to CI workflows sys::SE システムズエンジニアリング sys::SW ソフトウェア labels Sep 11, 2026
@shunsuke-shimomura
shunsuke-shimomura merged commit 85e71bd into main Sep 11, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci ⚙️ Changes to CI workflows documentation 📓 Improvements or additions to documentation sys::SE システムズエンジニアリング sys::SW ソフトウェア

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant