Skip to content

Fix badges in readme#9029

Merged
jacobtomlinson merged 1 commit intodask:mainfrom
fjetter:badges_readme
Mar 20, 2025
Merged

Fix badges in readme#9029
jacobtomlinson merged 1 commit intodask:mainfrom
fjetter:badges_readme

Conversation

@fjetter
Copy link
Member

@fjetter fjetter commented Mar 20, 2025

The current badges are broken

image

See https://docs.github.com/en/actions/monitoring-and-troubleshooting-workflows/monitoring-workflows/adding-a-workflow-status-badge for the documentation

I believe we're using a deprecated API here. These links should work (see rich diff)

Copy link
Member

@jacobtomlinson jacobtomlinson left a comment

Choose a reason for hiding this comment

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

Yeah things changed a while ago so you reference the YAML file path instead of the name of the workflow.

Thanks for fixing this!

@jacobtomlinson jacobtomlinson merged commit 4ef21ad into dask:main Mar 20, 2025
29 of 31 checks passed
@fjetter fjetter deleted the badges_readme branch March 20, 2025 11:49
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