ci(sync): remove hardcoded assignee/reviewer fallback - #348
Conversation
Use repository variables (or empty values) for SYNC_PR_ASSIGNEES and SYNC_PR_REVIEWERS so sync automation stays configurable and avoids a fixed username. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
|
Note Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported. |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Merging this PR will not alter performance
Comparing Footnotes
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Summary
Why
Follow-up CodeRabbit review on PR #345 requested that sync PR assignee/reviewer defaults not be tied to a fixed username.
Greptile Summary
このPRは、sync PRの自動生成時に
SYNC_PR_ASSIGNEESおよびSYNC_PR_REVIEWERSのフォールバック値としてハードコードされていたユーザー名is0692vsを削除し、空文字列''へ変更します。PR #345 に対する CodeRabbit のレビュー指摘を受けたフォローアップです。.github/workflows/sync-main-to-staging.ymlのSYNC_PR_ASSIGNEESとSYNC_PR_REVIEWERSのフォールバックを'is0692vs'→''に変更sync,automated) は汎用的なため引き続き維持sync-main-to-staging.sh内のparse_csv_values関数は空文字列を正しく処理するため、スクリプト側の変更は不要vars.SYNC_PR_ASSIGNEES/vars.SYNC_PR_REVIEWERS) が設定されていない場合、assignee・reviewer なしで PR が作成されるConfidence Score: 5/5
変更は最小限かつ安全で、マージに問題なし。
ハードコードされたユーザー名を空文字列へ置き換えるだけのシンプルな1行変更が2箇所。シェルスクリプト側の空値ハンドリング(parse_csv_values)も適切に実装されており、P1以上の問題は見当たらない。
特になし。
Important Files Changed
is0692vsフォールバックを削除し空文字列に変更。スクリプトの動作に影響なし。Sequence Diagram
sequenceDiagram participant GH as GitHub Actions participant WF as sync-main-to-staging.yml participant SH as sync-main-to-staging.sh participant API as GitHub API GH->>WF: push to main / workflow_dispatch WF->>WF: 環境変数を設定<br/>SYNC_PR_ASSIGNEES = vars.SYNC_PR_ASSIGNEES \|\| ''<br/>SYNC_PR_REVIEWERS = vars.SYNC_PR_REVIEWERS \|\| '' WF->>SH: bash スクリプト実行 SH->>API: 既存 PR を確認 alt 既存 PR なし かつ コミット差分あり SH->>API: gh pr create<br/>(変数が空の場合 --assignee / --reviewer はスキップ) else 既存 PR あり または 差分なし SH-->>GH: exit 0 endReviews (1): Last reviewed commit: "ci(sync): remove hardcoded sync assignee..." | Re-trigger Greptile