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

Actionsファイルの変更に対して警告をするactionsを追加する #21

Merged
merged 5 commits into from
Nov 28, 2023
Merged

Actionsファイルの変更に対して警告をするactionsを追加する #21

merged 5 commits into from
Nov 28, 2023

Conversation

umi1299
Copy link
Contributor

@umi1299 umi1299 commented Nov 24, 2023

内容

Actionsファイル (.github/workflows ディレクトリ以下) に変更のあるPRに対して警告のコメントをするactionsを作ってみました

関連

#17 (review)

Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

プルリクエストありがとうございます!!

やり方ですが、 tj-actions/changed-filesを使わずにon: pull_requestpathを使ったファイル検知とかもできそうだなと思いました!
checkoutやif: all_changed_files不要になってスッキリするかも?

あとgh pr commentpull_request_targetではなくpull_requestGITHUB_TOKENでもできそうでした。
(適当に検索して出てきたブログ記事
できることは同じですが権限が変わってきそうかなと・・・!

@umi1299
Copy link
Contributor Author

umi1299 commented Nov 27, 2023

確かに paths の使用でよかったですね
修正しました!

@umi1299
Copy link
Contributor Author

umi1299 commented Nov 27, 2023

トリガーを pull_request にしたらエラーになりました

GraphQL: Resource not accessible by integration (addComment)

https://github.com/VOICEVOX/homebrew-voicevox/actions/runs/6999903201/job/19039810943?pr=21

@umi1299 umi1299 requested a review from Hiroshiba November 27, 2023 02:55
Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!!

すごいシンプルなワークフローで良いですね!!


トリガーを pull_request にしたらエラーになりました

あれ、本当ですね・・・。シークレットも設定できてるし問題なさそうに見えるのですが・・・。
可能性としては

permissions:
  pull-requests: write

を書いてるから逆にエラーになる、とかでしょうか。。。。うーーーーーーーん・・・。
まあ少なくともこのプルリクエスト的には全く問題ないかなと思ったので、マージさせていただきます!

@Hiroshiba Hiroshiba merged commit e0fd783 into VOICEVOX:main Nov 28, 2023
5 checks passed
@umi1299 umi1299 deleted the feature-detect-actions-diff branch November 28, 2023 03:49
@umi1299
Copy link
Contributor Author

umi1299 commented Nov 28, 2023

フォークから作られたPRの場合、トリガーが pull_request_target でないactionsはpermissionsが記述してあってもシークレットのアクセス許可がすべてreadonlyになるので、それで失敗したのだと思います
貼っていただいたブログ記事ではリポジトリに対して、そのリポジトリの別ブランチからPRを作っていたので動いているのだと思います

@Hiroshiba
Copy link
Member

あーなるほどです!!!よくわかりました!!!!
詳しくありがとうございます、とても勉強になりました!!! 🙇

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.

2 participants