Skip to content

docs: show the state of main in the tests badge - #394

Merged
alexander-akait merged 1 commit into
mainfrom
docs/ci-badge-main
Oct 29, 2026
Merged

alexander-akait merged 1 commit into
mainfrom
docs/ci-badge-main

Conversation

@alexander-akait

@alexander-akait alexander-akait commented Oct 29, 2026 •

Copy link
Copy Markdown
Member

The tests badge in the README used the workflow-name URL (/workflows/diagnostics-webpack-plugin/badge.svg). That URL shows the workflow's latest run on any branch, so the badge turns red for runs that say nothing about main. Right now it is red for two such runs:

This switches to the workflow-file URL that webpack's own README uses (actions/workflows/test.yml/badge.svg). GitHub shows the default branch's status for that URL, and the badge link now opens this workflow's runs.

-[tests]: https://github.com/webpack/diagnostics-webpack-plugin/workflows/diagnostics-webpack-plugin/badge.svg
-[tests-url]: https://github.com/webpack/diagnostics-webpack-plugin/actions
+[tests]: https://github.com/webpack/diagnostics-webpack-plugin/actions/workflows/nodejs.yml/badge.svg
+[tests-url]: https://github.com/webpack/diagnostics-webpack-plugin/actions/workflows/nodejs.yml

The README change only, so there is no changeset.


Generated by Claude Code

Summary by CodeRabbit

  • Documentation
    • Updated the README’s test badge and link to point to the nodejs.yml GitHub Actions workflow. The displayed test status and the destination reached from the badge now correspond to that workflow. No other user-facing changes are included in this update.

The badge named the workflow, which shows its latest run on any branch:
a pull request's run, or one cancelled by the next push, turned it red.
Named by its file, as webpack's own README does, it shows the default
branch.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01App18sYbe6Vf6HDCjCXbvf
@linux-foundation-easycla

Copy link
Copy Markdown

CLA Not Signed

One or more co-authors of this pull request were not found. You must specify co-authors in commit message trailer via:

Co-authored-by: name <email>

Supported Co-authored-by: formats include:

  1. Anything <id+login@users.noreply.github.com> - it will locate your GitHub user by id part.
  2. Anything <login@users.noreply.github.com> - it will locate your GitHub user by login part.
  3. Anything <public-email> - it will locate your GitHub user by public-email part. Note that this email must be made public on Github.
  4. Anything <other-email> - it will locate your GitHub user by other-email part but only if that email was used before for any other CLA as a main commit author.
  5. login <any-valid-email> - it will locate your GitHub user by login part, note that login part must be at least 3 characters long.

Alternatively, if the co-author should not be included, remove the Co-authored-by: line from the commit message.

Please update your commit message(s) by doing git commit --amend and then git push [--force] and then request re-running CLA check via commenting on this pull request:

/easycla

@coderabbitai

coderabbitai Bot commented Oct 29, 2026

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 9530705f-1a4b-4140-90d1-a1c61829d6b8
Commits

Reviewing files that changed from the base of the PR and between b9bccba and 8c5b3e9.

Files selected for processing: 1
  • README.md

Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 1 remain after this review.

Walkthrough

The README test badge and its link now point to the nodejs.yml GitHub Actions workflow.

Priority: Low

Merge Risk: Minimal · up to 8c5b3

The documentation link change is safe to merge.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title "docs: show the state of main in the tests badge" matches the single main change: the README tests badge now points to the nodejs.yml workflow file, so it reflects the default branch (main) …
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Title check

Explanation

The title "docs: show the state of main in the tests badge" matches the single main change: the README tests badge now points to the nodejs.yml workflow file, so it reflects the default branch (main) status. The title is concise, uses a conventional prefix, and names the specific change. It is consistent with the objectives, which describe switching from the workflow-name URL to the workflow-file URL to show the default branch status.

Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

Finishing Touches: 💡 1
🛠️ Fix failing CI checks: 💡
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codecov

codecov Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.68%. Comparing base (b9bccba) to head (8c5b3e9).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #394   +/-   ##
=======================================
  Coverage   99.68%   99.68%           
=======================================
  Files          21       21           
  Lines        6450     6450           
=======================================
  Hits         6430     6430           
  Misses         20       20           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 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.

@alexander-akait
alexander-akait merged commit 998aa90 into main Oct 29, 2026
15 of 17 checks passed
@alexander-akait
alexander-akait deleted the docs/ci-badge-main branch October 29, 2026 12:29
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