Skip to content

Conversation

@y33550336
Copy link
Contributor

@y33550336 y33550336 commented Oct 14, 2025

一応ちゃんと見ましたが、AIに任せた部分が多いので正しいか分からないです
#1109

質問
・.yamlの名前はvalidate-renovate-config.yamlでいいですか
・jobsの中のvalidateはcheckがいいですか(参考資料がcheckになっている)
・actions/checkout@v4よりもactions/checkout@v3の方がいいですか(参考資料がv3になっている)
・今は対象ファイルを二つのステップで分けていますが一つの方がいいですか

@github-actions
Copy link

Diff

No diff detected!

@Kentaro1043 Kentaro1043 self-requested a review October 14, 2025 03:21
Comment on lines 6 to 7
- "renovate-bot.js"
- "renovate/**"
Copy link
Collaborator

Choose a reason for hiding this comment

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

パスが違う気がします

Suggested change
- "renovate-bot.js"
- "renovate/**"
- ".github/renovate-bot.js"
- ".github/renovate/**"

validate:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
Copy link
Collaborator

Choose a reason for hiding this comment

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

最新にするのが良いです
https://github.com/actions/checkout/releases

Suggested change
- uses: actions/checkout@v4
- uses: actions/checkout@v5

@Kentaro1043
Copy link
Collaborator

  • ファイル名は問題ないです
  • validateで大丈夫です、むしろ個人的にはcheckよりしっくりきます
  • 2つのステップに分けて大丈夫です

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