Skip to content

Point the build status badge at GitHub Actions - #1119

Open
tas50 wants to merge 1 commit into
chef:mainfrom
tas50:github_actions_build_badge
Open

Point the build status badge at GitHub Actions#1119
tas50 wants to merge 1 commit into
chef:mainfrom
tas50:github_actions_build_badge

Conversation

@tas50

@tas50 tas50 commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

The README build badge still pointed at the chef-oss/chef-cookstyle-main-verify Buildkite pipeline. CI runs on GitHub Actions now, so the badge showed the status of a pipeline that no longer gates this repo.

This swaps it for the unit workflow badge, filtered to main — that is the workflow that actually runs the test suite on every push to main. The link target uses ?query=branch%3Amain so clicking through lands on the main-branch runs rather than all runs.

-[![Build status](https://badge.buildkite.com/c086ffe05e32e4d61282b91ead96b3721590a59ed4360cf7ac.svg?branch=main)](https://buildkite.com/chef-oss/chef-cookstyle-main-verify)
+[![Unit tests](https://github.com/chef/cookstyle/actions/workflows/unit.yml/badge.svg?branch=main)](https://github.com/chef/cookstyle/actions/workflows/unit.yml?query=branch%3Amain)

Verification

  • Badge SVG and link target both return HTTP 200; the SVG currently renders unit - passing.
  • bundle exec rake — 1239 examples, 0 failures; lint and validate_config pass.

Left alone deliberately: there is also a lint workflow running on main that has no badge. Adding one is outside what this change is for, so flagging it rather than bundling it in.

The README's build badge still pointed at the chef-oss/chef-cookstyle-main-verify
Buildkite pipeline. CI moved to GitHub Actions, so the badge reflected a pipeline
that no longer gates this repo.

Swap it for the unit workflow, filtered to main, which is the workflow that
actually runs the test suite on every push to main. Verified the badge SVG and
its target both return 200 and that the badge currently renders 'unit - passing'.

Signed-off-by: Tim Smith <tsmith84@proton.me>
@tas50
tas50 requested review from a team and jaymzh as code owners August 24, 2026 18:40
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.

1 participant