Skip to content

Bugfix/cliff note format#121

Merged
yinheli merged 2 commits intodevelopfrom
bugfix/cliff-note-format
Jan 6, 2026
Merged

Bugfix/cliff note format#121
yinheli merged 2 commits intodevelopfrom
bugfix/cliff-note-format

Conversation

@yinheli
Copy link
Owner

@yinheli yinheli commented Jan 5, 2026

No description provided.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes the changelog format configuration in cliff.toml and updates several Rust dependencies. The changes simplify the group naming scheme by removing numeric prefixes and add conditional logic for contributor PR numbers.

Key Changes:

  • Simplified HTML comment markers in commit group names by removing numeric prefixes and spaces
  • Updated template processing to match the new group name format
  • Added conditional rendering for contributor PR numbers to handle cases where PR number may not be available

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.

File Description
cliff.toml Updated commit parser group names to remove numeric prefixes, adjusted template processing logic, and added conditional check for contributor PR numbers
Cargo.lock Routine dependency updates for clap, itoa, libc, portable-atomic, proc-macro2, ryu, syn, tokio, and related crates

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

cliff.toml Outdated
Comment on lines +3 to +13
{ message = "^feat", group = "<!--🚀 Features" },
{ message = "^fix", group = "<!--🐛 Bug Fixes" },
{ message = "^doc", group = "<!--📝 Documentation" },
{ message = "^perf", group = "<!--⚡ Performance" },
{ message = "^refactor", group = "<!--🏗️ Refactor" },
{ message = "^style", group = "<!--🎨 Styling" },
{ message = "^test", group = "<!--✅ Testing" },
{ message = "^chore\\(release\\): prepare for", skip = true },
{ message = "^chore\\(deps\\)", skip = true },
{ message = "^chore", group = "<!-- 7 -->⚙️ Miscellaneous Tasks" },
{ body = ".*security", group = "<!-- 8 -->🔒 Security" },
{ message = "^chore", group = "<!--⚙️ Miscellaneous Tasks" },
{ body = ".*security", group = "<!--🔒 Security" },
Copy link

Copilot AI Jan 5, 2026

Choose a reason for hiding this comment

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

The group names contain incomplete HTML comments that are missing the closing marker. The format <!--🚀 Features is not a valid HTML comment syntax - it should end with -->. If these are intended to be hidden HTML comments in the rendered changelog for sorting purposes, they need to be properly closed. If they are only internal markers for git-cliff, then the HTML comment syntax is unnecessary. Consider either using complete HTML comments like <!---->🚀 Features or using a different non-HTML marker format.

Copilot uses AI. Check for mistakes.
@yinheli yinheli force-pushed the bugfix/cliff-note-format branch from f21133a to 93f5128 Compare January 6, 2026 15:11
@yinheli yinheli self-assigned this Jan 6, 2026
@yinheli yinheli merged commit ed71bf4 into develop Jan 6, 2026
16 checks passed
@yinheli yinheli deleted the bugfix/cliff-note-format branch January 6, 2026 15:21
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