Conversation
|
Warning Rate limit exceeded@rickyheijnen has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 9 minutes and 41 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (1)
WalkthroughAdds a GitHub Actions workflow (.github/workflows/release.yml) that derives a version from composer.json, normalizes it to a v-prefixed tag, checks origin for the tag, and conditionally creates/pushes the tag and a GitHub Release; runs on pushes to main and manual dispatch. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Pre-merge checks and finishing touches✅ Passed checks (3 passed)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
.github/workflows/release.yml (1)
67-69: Limit git config scope to the repositoryUsing
git config --globalon a (likely persistent) self-hosted runner leaks these bot credentials into every job that reuses the machine. Please scope the identity to the checkout so we don’t mutate runner state between pipelines.- git config --global user.email "github-actions[bot]@users.noreply.github.com" - git config --global user.name "github-actions[bot]" + git config user.email "github-actions[bot]@users.noreply.github.com" + git config user.name "github-actions[bot]"
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
.github/workflows/release.yml(1 hunks)
🧰 Additional context used
🪛 actionlint (1.7.7)
.github/workflows/release.yml
18-18: label "brixion-runners" is unknown. available labels are "windows-latest", "windows-latest-8-cores", "windows-2025", "windows-2022", "windows-2019", "ubuntu-latest", "ubuntu-latest-4-cores", "ubuntu-latest-8-cores", "ubuntu-latest-16-cores", "ubuntu-24.04", "ubuntu-24.04-arm", "ubuntu-22.04", "ubuntu-22.04-arm", "ubuntu-20.04", "macos-latest", "macos-latest-xl", "macos-latest-xlarge", "macos-latest-large", "macos-15-xlarge", "macos-15-large", "macos-15", "macos-14-xl", "macos-14-xlarge", "macos-14-large", "macos-14", "macos-13-xl", "macos-13-xlarge", "macos-13-large", "macos-13", "self-hosted", "x64", "arm", "arm64", "linux", "macos", "windows". if it is a custom label for self-hosted runner, set list of labels in actionlint.yaml config file
(runner-label)
9d3904f to
a70ae79
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
.github/workflows/release.yml(1 hunks)
🧰 Additional context used
🪛 actionlint (1.7.7)
.github/workflows/release.yml
94-94: input "allow_updates" is not defined in action "softprops/action-gh-release@v2". available inputs are "append_body", "body", "body_path", "discussion_category_name", "draft", "fail_on_unmatched_files", "files", "generate_release_notes", "make_latest", "name", "prerelease", "preserve_order", "repository", "tag_name", "target_commitish", "token"
(action)
| - name: Create GitHub Release (autogenerated notes) | ||
| if: steps.release.outputs.exists == 'false' | ||
| uses: softprops/action-gh-release@v2 | ||
| with: | ||
| tag_name: ${{ steps.version.outputs.tag }} | ||
| name: Version ${{ steps.version.outputs.tag }} | ||
| generate_release_notes: true | ||
| allow_updates: true | ||
| env: |
There was a problem hiding this comment.
Remove unsupported allow_updates input
softprops/action-gh-release@v2 does not define an allow_updates input, so this step will fail at runtime with “Unexpected input 'allow_updates'” and the release will never be created.(github.com) Remove that key (the action already updates existing releases automatically) to keep the workflow green.
generate_release_notes: true
- allow_updates: true📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| - name: Create GitHub Release (autogenerated notes) | |
| if: steps.release.outputs.exists == 'false' | |
| uses: softprops/action-gh-release@v2 | |
| with: | |
| tag_name: ${{ steps.version.outputs.tag }} | |
| name: Version ${{ steps.version.outputs.tag }} | |
| generate_release_notes: true | |
| allow_updates: true | |
| env: | |
| - name: Create GitHub Release (autogenerated notes) | |
| if: steps.release.outputs.exists == 'false' | |
| uses: softprops/action-gh-release@v2 | |
| with: | |
| tag_name: ${{ steps.version.outputs.tag }} | |
| name: Version ${{ steps.version.outputs.tag }} | |
| generate_release_notes: true | |
| env: |
🧰 Tools
🪛 actionlint (1.7.7)
94-94: input "allow_updates" is not defined in action "softprops/action-gh-release@v2". available inputs are "append_body", "body", "body_path", "discussion_category_name", "draft", "fail_on_unmatched_files", "files", "generate_release_notes", "make_latest", "name", "prerelease", "preserve_order", "repository", "tag_name", "target_commitish", "token"
(action)
🤖 Prompt for AI Agents
In .github/workflows/release.yml around lines 87 to 95, remove the unsupported
allow_updates input from the softprops/action-gh-release@v2 step — delete the
allow_updates: true line so the action uses its built-in release update
behavior; keep the other inputs (tag_name, name, generate_release_notes)
unchanged and commit the change to prevent the “Unexpected input
'allow_updates'” runtime error.
654a37c to
fe7868c
Compare
4e8364b to
289d8aa
Compare
fe7868c to
41f9e1f
Compare
📝 Beschrijving
This PR adds a Github Action workflow for creating a tag release on every push to the
mainbranch.✅ Checklist