SerenityOS/serenity

PR Labeler: Does not properly relablel PRs

Open

#17,271 opened on 2023年2月2日

GitHub で見る
 (0 comments) (0 reactions) (0 assignees)C++ (33,171 stars) (3,328 forks)batch import
help wantedtask list

説明

We added the handy PR labeler CI action, but it doesn't seem to add labels properly in the following cases:

  • When a review is submitted by someone other than the author, change the state from "needs review" to "waiting for author feedback"
  • When a review is approved, move the state to "community approved" or "maintainer approved" based on who did the approval.
  • When a PR is detected to have conflicts, add the "has conflicts" label
  • When changes are pushed, remove the "waiting for author feedback" label and add "needs review"

コントリビューターガイド

技術スタック
github actionsshell
領域
ci cddevops
Issue 種別
bug
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
half day
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
GitHub ActionsYAML scripting
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
65
調査方針
Examine the existing PR labeler workflow in .github/workflows. Identify the current logic for labeling based on reviews, approvals, conflicts, and pushes. Determine where the failures occur for the listed cases. Consider using a community labeler action or custom script to handle all states. Check if there are any configuration files or environment variables that control labeling behavior.