Fix release notes to include all PR commits instead of only merge commit#51
Conversation
Co-authored-by: supervoidcoder <88671013+supervoidcoder@users.noreply.github.com>
…types, remove unused var Co-authored-by: supervoidcoder <88671013+supervoidcoder@users.noreply.github.com>
Co-authored-by: supervoidcoder <88671013+supervoidcoder@users.noreply.github.com>
|
@coderabbitai review |
WalkthroughModified release.yml to parse all PR commits from base to head instead of a single commit, categorizing each by conventional-commit type (feat, fix, perf, refactor, docs, test, chore, build, ci, style, revert, others), and generating structured multi-section release notes aggregating commits by category. Changes
Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 minutes Possibly related PRs
Suggested labels
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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 |
✅ Actions performedReview triggered.
|
supervoidcoder
left a comment
There was a problem hiding this comment.
great jjob just tiny nitpick
| } | ||
|
|
||
| if ($style.Count -gt 0) { | ||
| $releaseNotes += "### 💄 Style`n" |
There was a problem hiding this comment.
| $releaseNotes += "### 💄 Style`n" | |
| $releaseNotes += "### 🎨 Style`n" |
the lipstick emoji ticks me off lolllllllllllllllllllllllllllllllllllllllllllllllllllllllllllll
There was a problem hiding this comment.
@coderabbitai which other emojis you suggest for style?
There was a problem hiding this comment.
@supervoidcoder, here are some popular emoji alternatives for the Style category:
Most Common Choice:
- 🎨
:art:- Artist Palette - This is the standard in gitmoji and widely used for formatting/style changes
Other Good Options:
- 🧹
:broom:- Broom (your suggestion!) - Great for "cleaning up" code - 🖌️
:paintbrush:- Paintbrush - Similar vibe to the palette - 📐
:triangular_ruler:- Triangular Ruler - Emphasizes alignment/formatting - 📏
:straight_ruler:- Straight Ruler - Also works for formatting - 🧼
:soap:- Soap - Cleaning/tidying up code
My recommendation: Go with 🎨 (artist palette) as it's the most recognized in the developer community for style/formatting commits, or stick with your 🧹 (broom) suggestion if you prefer the "cleaning up code" metaphor!
🐰✨
🧠 Learnings used
Learnt from: supervoidcoder
Repo: supervoidcoder/win-witr PR: 48
File: README.md:38-38
Timestamp: 2026-01-21T01:52:33.870Z
Learning: For supervoidcoder's README files in the win-witr repository: The user prefers an intentionally informal, humorous tone with casual language. Grammar formality suggestions (like "anyways" vs "anyway") should be skipped as they don't align with the intentional style of the documentation.
There was a problem hiding this comment.
ok great i did the palette
There was a problem hiding this comment.
@supervoidcoder Awesome! 🎨 is a solid choice – it's the standard for style/formatting changes across the dev community. Your release notes will look great!
🐰✨
🧠 Learnings used
Learnt from: supervoidcoder
Repo: supervoidcoder/win-witr PR: 48
File: README.md:38-38
Timestamp: 2026-01-21T01:52:33.870Z
Learning: For supervoidcoder's README files in the win-witr repository: The user prefers an intentionally informal, humorous tone with casual language. Grammar formality suggestions (like "anyways" vs "anyway") should be skipped as they don't align with the intentional style of the documentation.
If you found this review helpful, would you consider giving us a shout-out on X?
Thank you for using CodeRabbit!
Release workflow was only parsing the merge commit message (
git log -1), omitting all individual commits from PRs.Changes
git log $baseSha..$headShato retrieve all commits between PR base and headBefore/After
Before:
After:
The workflow now surfaces actual changes rather than merge commit metadata.
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.