Changelog automation fixes#8988
Conversation
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ 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. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe PR extends the changelog generation workflow and script to accept ChangesChangelog Workflow and Script Updates
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
This PR updates the changelog generation workflow to collect additional release metadata for generated Mattermost documentation changelog entries.
Changes:
- Adds a
release_typeworkflow input withfeatureandesroptions. - Adds a
go_versionworkflow input. - Passes both new inputs into the changelog generation step environment.
|
Newest code from mattermost has been published to preview environment for Git SHA 0d80bc1 |
|
Newest code from mattermost has been published to preview environment for Git SHA e0c943d |
|
Newest code from mattermost has been published to preview environment for Git SHA 49ce559 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
|
@coderabbitai full review |
✅ Actions performedFull review triggered. |
|
Newest code from mattermost has been published to preview environment for Git SHA a348f29 |
|
Newest code from mattermost has been published to preview environment for Git SHA 5809bff |
|
Newest code from mattermost has been published to preview environment for Git SHA ab4f6d5 |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.github/scripts/generate_changelog.py:
- Around line 356-362: The current replacement only swaps the "### Go Version"
heading and can leave Claude's body text intact; update the logic that uses the
polished and go_section variables so when "### Go Version" exists you replace
the entire section from the "### Go Version" heading up to (but not including)
the next "### " heading or end-of-string in one operation (use a single regex
replace for the whole section) and otherwise append go_section as before.
- Line 345: The go_section assignment uses an unnecessary f-string literal;
update the go_section variable (symbol: go_section) to use a plain string
literal (remove the leading "f") since there are no placeholders, ensuring the
line becomes a normal string to satisfy Ruff F541.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: cf28a56c-0b4c-4e75-b234-4ffebbdabec4
📒 Files selected for processing (2)
.github/scripts/generate_changelog.py.github/workflows/generate-changelog.yml
|
Newest code from mattermost has been published to preview environment for Git SHA 6aa3881 |
|
Newest code from mattermost has been published to preview environment for Git SHA 2d56bd7 |
|
Newest code from mattermost has been published to preview environment for Git SHA 04af83e |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.github/scripts/generate_changelog.py:
- Around line 355-366: The current else branch always appends go_section to
polished, which puts the Go Version at the end even when later top-level
headings like "### Security" or "### Open Source Components" exist; update the
logic so that if no "### Go Version" heading is found but polished contains a
later top-level section (match e.g. r"(?m)^### (Security|Open Source
Components)\b"), insert go_section immediately before the first such matched
heading, otherwise fall back to appending to the end; reference the variables
polished and go_section and the existing regex search for "### Go Version" to
locate where to modify the insertion behavior.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 3ddab4aa-8632-441c-ab9d-65588686b22b
📒 Files selected for processing (1)
.github/scripts/generate_changelog.py
|
Newest code from mattermost has been published to preview environment for Git SHA 44329b5 |
Minor improvements to the changelog generation automation.
go1.22.5.) or falls back to "Go version is the same as in the previous release.". The section is injected by replacing the AI's ### Go Version placeholder, or appended if the AI omitted it.release-noteblocks. If no#### Release Notesection is found, the script now picks up anyrelease-noteblocks wherever they appear, matching the format you showed.merged_at: For any PR where merged_at is unexpectedly null, the script now makes a direct call to the Pulls API (/repos/{owner}/{repo}/pulls/{number}) to verify the merged flag before skipping it.requests.get()calls with a newmake_github_request()helper that reads theX-RateLimit-Resetheader and sleeps until the window resets before retrying, up to 3 retries.