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

自動生成したプルリクエストでactionが発火しない #194

Closed
korosuke613 opened this issue May 1, 2020 · 6 comments · Fixed by #201
Closed

自動生成したプルリクエストでactionが発火しない #194

korosuke613 opened this issue May 1, 2020 · 6 comments · Fixed by #201
Assignees
Labels
bug Something isn't working

Comments

@korosuke613
Copy link
Member

起こっている問題 / The Problem

スクリーンショット / Screenshot

80787188-d0b92e00-8bc0-11ea-97bc-0c2c8934225e
#192

期待する見せ方・挙動 / Expected Behavior

  • create-pull-requestを使ってプルリクエストを自動生成した際に、pr_test.ymlが動く。

起こっている問題の再現手段 / Steps to Reproduce

create-pull-requestを使ってプルリクエストを作成してください。

参考:
#192 , #148

@korosuke613 korosuke613 added the bug Something isn't working label May 1, 2020
@korosuke613
Copy link
Member Author

とりあえず対応します。

@korosuke613 korosuke613 self-assigned this May 1, 2020
@korosuke613
Copy link
Member Author

同じ問題についてcreate-pull-requestのissueにありました。
どうやら、アクションが他のワークフローをトリガーすることはできないみたいです。
回避策もあるみたいなので、また何かわかったらここに書きます。

@peter-evans
Copy link

いくつかの方法があるんですが、一番簡単なのはPAT(repo scoped)を作って、レポのsecretとして追加して、アクションのtokenに設定します。

    - name: Create Pull Request
      id: cpr
      uses: peter-evans/create-pull-request@v2
      with:
+       token: ${{ secrets.PAT }}
        commit-message: 'Update news.json'
        branch: news
        base: master
        labels: auto_merge
        branch-suffix: timestamp
        title: '最新のお知らせ更新'
        body: '公式サイトのお知らせが更新されました。差分を確認してマージしてください。'

@korosuke613
Copy link
Member Author

Dear @peter-evans

コメントありがとうございます。(しかも日本語で😭)
あなたのページを読み、4つの方法を実験しました。
私もPATを使う方法が一番楽で便利だと考えます。ただ、自分のアカウントでapprove, request changesできないことが不便です。
なので、あなたのページに載っているように、新しいmachine userを作成し、そのPATを利用する予定です。

"create-pull-request"はとても便利です。"advanced usage"もとても参考になりました。
ありがとうございました🙇😆

↓↓↓↓↓↓↓↓english version(using DeepL)↓↓↓↓↓↓↓↓

Thank you for your comment. (and in Japanese😭)
I've read your page and experimented with four ways.
I also find the PAT method to be the easiest and most convenient. However, it is inconvenient that you cannot approve, request changes in your account.
So I'm going to create a new machine user and use that PAT, as shown on your page.

"Create-pull-request" is very useful. And "advanced usage" was also very helpful.
Thank you very much🙇😆

@korosuke613
Copy link
Member Author

こちらの問題ですが、今日中に対応します。

@korosuke613
Copy link
Member Author

image

いろいろ試した結果、新たなmachine userを作って、Personal Access Token(PAT)を発行し、create-pull-requestにそのPATを使わせることでワークフローで作ったプルリクエストがワークフローを発火できることがわかりました。

    - uses: peter-evans/create-pull-request@v2.7.0
      name: Create Pull Request
      with:
        token: ${{ secrets.BOT_TOKEN }}

secretsに保存したPATをcreate-pull-requestのtokenに設定することで実現できます。

作成したPATは本リポジトリのsecretに設定しておきます。

korosuke613 added a commit that referenced this issue May 4, 2020
closed #194 自動生成したプルリクエストでテストが走るようにした
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
2 participants