Skip to content

chore: Re-apply the repo-ci-baseline Template - #258

Merged
mpaulosky merged 1 commit into
mainfrom
chore/reapply-baseline
Oct 10, 2026
Merged

mpaulosky merged 1 commit into
mainfrom
chore/reapply-baseline

Conversation

@mpaulosky

Copy link
Copy Markdown
Owner

Why

Re-Apply the repo-ci-baseline Template so this repo carries the Template changes from mpaulosky/dotfiles#185.

What changed

Verification

  • The Apply commit is exactly apply.sh's output on a branch up to date with main.
  • The pre-push gate (scripts/gate.sh) passed on f3c1ca2.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@codecov

codecov Bot commented Oct 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.63%. Comparing base (d69fb63) to head (f3c1ca2).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #258   +/-   ##
=======================================
  Coverage   85.63%   85.63%           
=======================================
  Files          75       75           
  Lines        1566     1566           
  Branches      186      186           
=======================================
  Hits         1341     1341           
  Misses        190      190           
  Partials       35       35           
🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions

Copy link
Copy Markdown
Contributor

Test Results Summary

485 tests  ±0   485 ✅ ±0   22s ⏱️ -5s
  9 suites ±0     0 💤 ±0 
  9 files   ±0     0 ❌ ±0 

Results for commit f3c1ca2. ± Comparison against base commit d69fb63.

@mpaulosky mpaulosky added the review:claude Claude reviews each head, a backup to Copilot's review label Oct 10, 2026

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Claude Review

The PR changes only codecov.yml. It replaces coverage.status.patch: true and project: true with default statuses marked informational: true, plus a comment explaining why. Codecov will still report patch and project coverage on every PR, but those checks always pass. The coverage gate is left to ci.yml's threshold check; CLAUDE.md says CI fails below 60% line coverage. This stops Codecov's base-relative targets (effectively 100% on a fully covered repo) from failing a check that PR Auto-Merge would treat as blocking.

The new YAML is valid Codecov syntax at the right level, alongside the existing changes: false and default_rules. Codecov posts informational statuses as success, so they can't block a merge, and nothing else in the config changes.

Things I'm relying on but couldn't check here:

  • Codecov's documented behaviour that informational: true always reports success, including when no upload arrives.
  • That ci.yml does enforce .github/ci/coverage-threshold as the comment says. I didn't open ci.yml or that threshold file.
  • That codecov.yml is managed by the repo-ci-baseline Template. The head commit is a Template Apply, so I assume this change comes from the Template; if it doesn't, the next Apply could overwrite it.

The only diff piece, 001.diff (codecov.yml), was read in full; no files were excluded or left unreviewed.

Verdict: no problems found. Safe to merge.

No findings.

@mpaulosky
mpaulosky merged commit 9f88d73 into main Oct 10, 2026
36 checks passed
@mpaulosky
mpaulosky deleted the chore/reapply-baseline branch October 10, 2026 21:55
mpaulosky added a commit that referenced this pull request Oct 10, 2026
Automated release blog posts for #258, opened by the release workflow.
It holds every Release whose post isn't on main yet, rebuilt from main
on each run. The [skip-release] title marker keeps its merge from
starting another release.

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review:claude Claude reviews each head, a backup to Copilot's review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant