Skip to content

chore(docs): fix CI badge status display#1294

Merged
styfle merged 2 commits intovercel:mainfrom
MikeMcC399:fix/ci-badge
Oct 27, 2025
Merged

chore(docs): fix CI badge status display#1294
styfle merged 2 commits intovercel:mainfrom
MikeMcC399:fix/ci-badge

Conversation

@MikeMcC399
Copy link
Copy Markdown
Contributor

@MikeMcC399 MikeMcC399 commented Oct 26, 2025

Situation

The CI badge is showing an incorrect failing status.

image

The badge for the CI workflow .github/workflows/ci.yml, displayed in the readme.md, shows "CI failing", although the action last ran successfully in the default main branch.

The badge is using the URL:

https://github.com/vercel/ncc/workflows/CI/badge.svg

The syntax of this URL is not in line with the GitHub documentation Adding a workflow status badge which specifies the format:

https://github.com/OWNER/REPOSITORY/actions/workflows/WORKFLOW-FILE/badge.svg

Change

In the readme.md change the URL to

https://github.com/vercel/ncc/actions/workflows/ci.yml/badge.svg

Verification

Check the readme.md and confirm that the badge status conforms to the status of the latest action run in the default main branch.

Currently this should be showing

image

@MikeMcC399

This comment was marked as outdated.

@MikeMcC399 MikeMcC399 marked this pull request as ready for review October 26, 2025 07:00
@styfle styfle changed the title docs: fix CI badge status display chore(docs): fix CI badge status display Oct 27, 2025
@styfle styfle enabled auto-merge (squash) October 27, 2025 15:25
Copy link
Copy Markdown
Member

@styfle styfle left a comment

Choose a reason for hiding this comment

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

Thanks!

@styfle styfle merged commit 1b764a8 into vercel:main Oct 27, 2025
11 checks passed
@MikeMcC399 MikeMcC399 deleted the fix/ci-badge branch October 27, 2025 15:39
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