Skip to content

refactor: separate submit-work and finish-work commands#13

Merged
glassmonkey merged 31 commits intomainfrom
fix/skip-pr-create-prompt
Feb 10, 2025
Merged

refactor: separate submit-work and finish-work commands#13
glassmonkey merged 31 commits intomainfrom
fix/skip-pr-create-prompt

Conversation

@glassmonkey
Copy link
Copy Markdown
Owner

@glassmonkey glassmonkey commented Feb 10, 2025

作業フローの改善

概要

作業フローを改善し、コマンドの実行をMakefile経由に統一します。

変更内容

  • gh pr createの対話的なプロンプトをスキップするように変更
    • --repoオプションを使用して、リポジトリの指定を自動化
  • 作業完了手順を分割
    • submit-work: PRの作成を担当
    • finish-work: レビュー完了後の作業を担当
  • レビュー監視の改善
    • monitor-prターゲットを追加
    • CIとレビューの状態を監視
    • スクリプトの直接呼び出しをMakefile経由に変更
  • ルールの明確化
    • 各コマンドの役割と実行タイミングを明確化
    • PRマージ後の後始末手順を追加
    • コマンドの実行をMakefile経由に統一

確認事項

  • make submit-work title="<pr-title>"でPRが作成できること
  • make monitor-prでPRの状態が監視できること
  • make finish-workでPRマージ後の後始末が実行できること

補足

  • PRマージ後は必ずmake finish-workを実行してください
    • .workディレクトリの掃除
    • mainブランチへの切り替え
    • 作業ブランチの削除
      を実行します。

@glassmonkey glassmonkey changed the title fix: skip gh pr create prompt refactor: improve workflow and unify command execution Feb 10, 2025
@glassmonkey glassmonkey changed the title refactor: improve workflow and unify command execution refactor: separate submit-work and finish-work commands Feb 10, 2025
@glassmonkey
Copy link
Copy Markdown
Owner Author

LGTM

@glassmonkey
Copy link
Copy Markdown
Owner Author

LGTM

1 similar comment
@glassmonkey
Copy link
Copy Markdown
Owner Author

LGTM

@glassmonkey
Copy link
Copy Markdown
Owner Author

LGTM

@glassmonkey
Copy link
Copy Markdown
Owner Author

LGTM

@glassmonkey
Copy link
Copy Markdown
Owner Author

LGTM

@glassmonkey
Copy link
Copy Markdown
Owner Author

LGTM

@glassmonkey glassmonkey merged commit 3093873 into main Feb 10, 2025
@glassmonkey glassmonkey deleted the fix/skip-pr-create-prompt branch February 10, 2025 07:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant