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/checkout@v3 を使う #122

Closed
wants to merge 1 commit into from

Conversation

uchinokot
Copy link

目的

ku00
ku00 previously approved these changes Sep 16, 2022
Copy link
Contributor

@ku00 ku00 left a comment

Choose a reason for hiding this comment

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

試しにApproveしてみてCIが動くかみてみる

https://docs.github.com/ja/repositories/managing-your-repositorys-settings-and-features/enabling-features-for-your-repository/managing-github-actions-settings-for-a-repository#controlling-changes-from-forks-to-workflows-in-public-repositories

パブリックリポジトリをフォークし、リポジトリのGitHub Actionsワークフローへの変更を提案するPull Requestをサブミットすることは誰でもできます。 フォークからのワークフローはシークレットなどの機密データにアクセスできませんが、悪用目的で変更された場合、メンテナが迷惑を被る可能性があります。

これを防ぐために、外部コラボレータのパブリックリポジトリへのPull Requestではワークフローは自動的には動作せず、まず承認が必要になることがあります。 デフォルトでは、すべての初めてのコントリビューターは、ワークフローを実行するのに承認を必要とします。

@ku00 ku00 dismissed their stale review September 16, 2022 10:23

ApproveしてもCIは動かなかった

@ku00 ku00 mentioned this pull request Sep 16, 2022
Copy link
Contributor

@ku00 ku00 left a comment

Choose a reason for hiding this comment

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

再度承認

@ku00
Copy link
Contributor

ku00 commented Sep 20, 2022

on: pull_request を指定すれば動くのでは〜という話をしました
https://docs.github.com/ja/actions/using-workflows/events-that-trigger-workflows#pull_request

@ken3ypa
Copy link
Contributor

ken3ypa commented Sep 20, 2022

こちらのPRで on: pull_request をトリガーに追加する差分を入れようとするも、fork運用でPR作成しているため権限なくで弾かれる & 内野さん本日不在のため、試すにはPR別に作る必要がありそうですね 🙏

❯ git push
ERROR: Permission to uchinokot/active_merchant-epsilon.git denied to ken3ypa.
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

@ken3ypa
Copy link
Contributor

ken3ypa commented Sep 20, 2022

#124actions/checkout@v3 にせず、on: pull_request のみの差分にしたPR作成したところ、CIが走ることを確認しました 🎉

@ku00
Copy link
Contributor

ku00 commented Sep 20, 2022

検証が完了したのでこちらのPRはクローズします

@ku00 ku00 closed this Sep 20, 2022
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.

3 participants