Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

release → main プルリクエストのマージをトリガーとして release → develop へマージする #402

Closed
takada-masaya opened this issue Mar 9, 2023 · 5 comments
Assignees

Comments

@takada-masaya
Copy link

takada-masaya commented Mar 9, 2023

まずはじめに、素晴らしいアクションを提供いただき、ありがとうございます。

yanamura/git-flow-merge-action を使用するにあたって、
サンプルで提供されているようなラベルの付与ではなく、
releasemain のプルリクエストがマージをされたのをトリガーとして
releasedevelop のマージを自動で行いたいと考えています。

on:
  pull_request:
    branches:
      - main
    types: [closed]
jobs:
  auto-merge:
    if: github.event.pull_request.merged == true
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v3
        name: Checkout repository
      - name: Extract branch name
        uses: mdecoleman/pr-branch-name@2.0.0
        id: extract_branch
        with:
          repo-token: ${{ secrets.GITHUB_TOKEN }}
      - uses: yanamura/git-flow-merge-action@v1.2.0
        name: git-flow merge
        with:
          github_token: ${{ secrets.GITHUB_TOKEN }}
          branch: ${{ steps.extract_branch.outputs.branch }}
          develop_branch: "develop"
          main_branch: "main"

上記のような設定で実際にプルリクエストのマージを試してみると、
releasedevelop のマージは行われるのですが、
以下のような警告が発生します。
image

これらの警告は無視しても問題ないのか、
問題がある場合はどのような改善策があるか、
ご教授いただけますと幸いです。
よろしくお願いいたします。

@takada-masaya takada-masaya changed the title ぷる release → main プルリクエストのマージをトリガーとして release → develop へマージする Mar 9, 2023
@yanamura yanamura self-assigned this Apr 10, 2023
@yanamura
Copy link
Owner

@takada-masaya 反応遅くなり申し訳ありません。

現時点の実装では、mainとdevelopに指定したブランチをマージするようにしてしまっているので、mainにマージ済みだとエラーになってしまっていると思われます。修正すれば対応することは可能ですが、少々お時間いただくことになります。

代替案としましては、mainにPRをマージする際にGitHub上でマージするのではなく、こちらの記事のように特定のラベルを付与をトリガーにしていただくとちゃんと動作すると思います。
https://tech.connehito.com/entry/2020/09/30/174451

warningに関してはこちらで修正してリリースします

@takada-masaya
Copy link
Author

@yanamura
ご返信ありがとうございます。

ラベルの付与をトリガーにすればエラーが発生しないことは確認しておりますが、
社内の運用ルールで、GitHub上でマージをすることになっていますので、
一度ラベルの付与でマージをすることを検討してみます。

現状エラーは表示されていますがdevelopへの自動マージは問題なく行えていますので、
お手好きの際にマージ済みブランチのマージについてご確認いただけますと幸いです。

よろしくお願いいたします。

@yanamura
Copy link
Owner

@takada-masaya v1.4.0でwarningとマージ済みのbranchにマージでエラーにならないように対応してみましたので、ご確認いただけたらと思います。
https://github.com/yanamura/git-flow-merge-action/releases/tag/v1.4.0

@takada-masaya
Copy link
Author

@yanamura
ご修正ありがとうございます。

こちらの環境でエラー、警告どちらも表示されないことを確認しました!
今後ともお世話になりますが、よろしくお願いいたします。

@yanamura
Copy link
Owner

@takada-masaya 解決してよかったです!

こちらクローズしますね

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

No branches or pull requests

2 participants