Skip to content

Polish Commit Message Checklist - #1081

Draft
papeh wants to merge 1 commit into
mainfrom
feature/polish-commit-template
Draft

Polish Commit Message Checklist#1081
papeh wants to merge 1 commit into
mainfrom
feature/polish-commit-template

Conversation

@papeh

@papeh papeh commented Aug 18, 2026

Copy link
Copy Markdown
Contributor
  • Remove check for whitespace: this is redundant to automatic checks
  • Add a step to self-review with AI per our team decision

TODO: ensure commit message line length guidelines match commit hooks (and remove from human checks what the machine already does automatically)

CI-ready checklist

  • Commit messages follow .github/commit-guidelines.md (subject ≤ 72 chars, no trailing punctuation; if body present, blank line then ≤ 80-char lines).
  • Builds/tests pass locally (or I've run the CI-style build via build.ps1/test.ps1 or MSBuild).
  • If this is core-developer AI-assisted work, I followed Docs/workflows/ai-pr-workflow.md and ran pr-preflight or the equivalent branch-readiness review before requesting review.
  • For any Src/** folders touched, corresponding AGENTS.md files are updated or explicitly confirmed still accurate.
  • Devin has been appeased

This change is Reviewable

* Remove check for whitespace: this is redundant to automatic checks
* Add a step to self-review with AI per our team decision

TODO: ensure commit message line length guidelines match commit hooks
@github-actions

Copy link
Copy Markdown

NUnit Tests

    1 files  ±0      1 suites  ±0   8m 11s ⏱️ - 2m 0s
5 780 tests ±0  5 699 ✅ ±0  81 💤 ±0  0 ❌ ±0 
5 789 runs  ±0  5 708 ✅ ±0  81 💤 ±0  0 ❌ ±0 

Results for commit 3ce292f. ± Comparison against base commit ec2ab0a.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 38.07%. Comparing base (ec2ab0a) to head (3ce292f).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1081   +/-   ##
=======================================
  Coverage   38.07%   38.07%           
=======================================
  Files        1499     1499           
  Lines      350154   350154           
  Branches    40241    40241           
=======================================
+ Hits       133306   133314    +8     
+ Misses     187568   187559    -9     
- Partials    29280    29281    +1     

see 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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