Skip to content

Require unit tests in the pull request template - #1104

Open
papeh wants to merge 1 commit into
mainfrom
polish/commit-checklist
Open

Require unit tests in the pull request template#1104
papeh wants to merge 1 commit into
mainfrom
polish/commit-checklist

Conversation

@papeh

@papeh papeh commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

CI-ready checklist


This change is Reviewable

@papeh
papeh marked this pull request as ready for review August 21, 2026 17:06
@github-actions

github-actions Bot commented Aug 21, 2026

Copy link
Copy Markdown

NUnit Tests

    1 files  ±0      1 suites  ±0   11m 10s ⏱️ +10s
5 853 tests ±0  5 772 ✅ ±0  81 💤 ±0  0 ❌ ±0 
5 862 runs  ±0  5 781 ✅ ±0  81 💤 ±0  0 ❌ ±0 

Results for commit 4a4cc47. ± Comparison against base commit 6f72671.

♻️ This comment has been updated with latest results.

@codecov-commenter

codecov-commenter commented Aug 21, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 38.36%. Comparing base (6f72671) to head (4a4cc47).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1104      +/-   ##
==========================================
+ Coverage   38.33%   38.36%   +0.02%     
==========================================
  Files        1507     1507              
  Lines      350580   350580              
  Branches    40293    40293              
==========================================
+ Hits       134410   134501      +91     
+ Misses     186941   186900      -41     
+ Partials    29229    29179      -50     

see 10 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