Skip to content

fix: replace broken badge URLs with GitHub-native format#55

Closed
cdprice02 wants to merge 1 commit intomainfrom
fix/issue-52-readme-banners
Closed

fix: replace broken badge URLs with GitHub-native format#55
cdprice02 wants to merge 1 commit intomainfrom
fix/issue-52-readme-banners

Conversation

@cdprice02
Copy link
Copy Markdown
Owner

Description

Closes #52.

Replaces broken github-actions-badge.deno.dev badge URLs with GitHub's native badge format. The separate build and test badges (which both reflected the same workflow) are consolidated into a single workflow status badge. The lint badge is updated similarly. The Coveralls coverage badge is unchanged.

Type of change

  • Bug fix

Test checklist

  • cargo test passes
  • cargo clippy --all-targets --all-features -- -D warnings passes

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings April 8, 2026 01:10
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the project README badges to use GitHub’s native workflow badge URLs to avoid broken third-party badge images and reduce badge duplication.

Changes:

  • Replace github-actions-badge.deno.dev URLs with GitHub-native actions/workflows/.../badge.svg URLs.
  • Consolidate the prior build/test badges into a single workflow status badge.
  • Update the lint badge URL to the same GitHub-native format.

Comment on lines +3 to +5
[![build](https://github.com/cdprice02/ferrish/actions/workflows/ci.yml/badge.svg?branch=main)](https://github.com/cdprice02/ferrish/actions/workflows/ci.yml)
[![coverage](https://coveralls.io/repos/github/cdprice02/ferrish/badge.svg?branch=main)](https://coveralls.io/github/cdprice02/ferrish?branch=main)
[![lint](https://github-actions-badge.deno.dev/cdprice02/ferrish/ci.yml?label=lint)](https://github.com/cdprice02/ferrish/actions/workflows/ci.yml)
[![lint](https://github.com/cdprice02/ferrish/actions/workflows/ci.yml/badge.svg?branch=main)](https://github.com/cdprice02/ferrish/actions/workflows/ci.yml)
Copy link

Copilot AI Apr 8, 2026

Choose a reason for hiding this comment

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

Both the build and lint badges now point to the same GitHub Actions workflow badge (ci.yml/badge.svg), so they will always display identical status. This is misleading (the workflow badge is not job-specific) and creates redundant badges; consider replacing both with a single CI badge, or introducing a separate workflow for lint if you want a distinct lint badge.

Copilot uses AI. Check for mistakes.
@cdprice02
Copy link
Copy Markdown
Owner Author

Closing — this replaced the deno badges with a single GitHub-native badge, losing per-job granularity. Will reopen with correct deno badge labels and a CI push trigger for coverage.

@cdprice02 cdprice02 closed this Apr 8, 2026
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.

Fix README banners

2 participants