Skip to content

Conversation

@fffk10
Copy link
Member

@fffk10 fffk10 commented Nov 4, 2025

issue

なし

GitHub Actions pinningの件に関連
組織横断でworkflowを必須にできるかを調査するために、お試し用のworkflowを追加
(調査終わったら消します)

変更内容

  • お試し用のworkflowを追加
    • これをorgのrulesetで指定して組織横断で強制できるかを試します
      ※見たところ、組織横断でworkflowを必須にするにはEnterprise契約が必要だという記述を見ましたが、少し古い記事しか見つからなかったので本当か試してみます
      ※個人のorgでも契約的にruresetの適用ができませんでした

スレッドにもメモを残しています
https://ncdctalk.slack.com/archives/C097AFZ36H1/p1762297379289669?thread_ts=1761732211.579479&cid=C097AFZ36H1

確認したこと

スクリーンショット

補足事項

PR時のセルフチェック

  • 関数や変数の命名は一目で分かるものになってますか?
  • コメントは適切な量で、誰が見ても分かるコメントになってますか?
  • PR内容に関するテストは書きましたか?
  • PR内容に付随する各種ドキュメントは一緒に修正しましたか?
  • IssueをCloseする場合、完了の定義は満たせていますか?

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@fffk10
Copy link
Member Author

fffk10 commented Nov 4, 2025

internalで確認依頼する

@fffk10 fffk10 marked this pull request as draft November 4, 2025 23:52
@fffk10 fffk10 marked this pull request as ready for review November 5, 2025 01:34
@fffk10 fffk10 requested a review from eai04191 November 5, 2025 01:34
@eai04191 eai04191 merged commit b82270e into main Nov 5, 2025
1 check passed
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