Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: fix status badges in readme #1143

Merged
merged 1 commit into from
Mar 8, 2024

Conversation

MikeMcC399
Copy link
Collaborator

@MikeMcC399 MikeMcC399 commented Mar 8, 2024

Issue

The badge for the newly-added workflow .github/workflows/example-start-and-pnpm-workspaces.yml shows "no status", instead of the expected "passed" status in the README > pnpm workspaces section.

Changes

Most status badges in the README do not conform to the documentation Adding a workflow status badge which specifies the format

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

OWNER = cypress-io
REPOSITORY = github-action
WORKFLOW-FILE = e.g. example-start-and-pnpm-workspaces.yml (including the .yml extension)

In all badges, where necessary,

  1. change /github-action/workflows/ to /github-action/actions/workflows/
  2. add .yml extension where missing
  3. for simplification remove ?branch=master since this is unnecessary. It is the default setting.

Verification

Scroll through the README file and check that all status badges are showing a status of "passed".

@cypress-app-bot
Copy link

@MikeMcC399 MikeMcC399 added documentation Improvements or additions to documentation ci labels Mar 8, 2024
@MikeMcC399
Copy link
Collaborator Author

@MikeMcC399 MikeMcC399 marked this pull request as ready for review March 8, 2024 08:12
Copy link
Member

@jennifer-shehane jennifer-shehane left a comment

Choose a reason for hiding this comment

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

Thanks!

@jennifer-shehane jennifer-shehane merged commit cb48363 into cypress-io:master Mar 8, 2024
76 checks passed
@MikeMcC399 MikeMcC399 deleted the fix/status-badges branch March 8, 2024 12:18
Copy link

🎉 This PR is included in version 6.7.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci documentation Improvements or additions to documentation released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

pnpm workspaces workflow shows "no status"
3 participants