Remove broken badge#4121
Conversation
abfa242 to
56fe8c1
Compare
|
what is the point of this badge? just conveying that tests are passing? maybe this workflow is right for that: https://github.com/zarr-developers/zarr-python/actions/workflows/test.yml |
|
and if you have the patience for it, we could do a horizontal layout for these badges to save space. See the example from pydantic's readme |
|
It did convey that the .github/workflows/python-package.yml workflow, combining liniting, testing and building was passing. If you ask me, it's best to get rid of it. OK for the horizontal layout. |
👍 |
It relies on a CI workflow that had been removed in 2024 by 9e3e8c0. It's unclear whether it should be replaced by a different CI workflow, and if so which one.
56fe8c1 to
9fdba08
Compare
|
Changes as follows:
I have removed the first column of the vertical tabular layout, as it doesn't fit the horizontal layout, doesn't convey crucial or new information, and is not standard (badges shouldn't need an explanation or title). The line remains a bit long, despite I removed 2 badges. How do you like it? |
ca75df4 to
c9529ad
Compare
c9529ad to
df1cb39
Compare
Summary
It relies on a CI workflow (
.github/workflows/python-package.yml) that had been removed in 2024 by 9e3e8c0.It's unclear whether it should be replaced by a different CI workflow, and if so which one.
For reviewers
Would you rather replace with a different workflow? Which one?
Author attestation
AI coding assistance is welcome, but a human must be the author and is responsible for the contents of the PR. The description and any review responses must be in your own words. Please read AI-assisted contributions before opening.
TODO
docs/user-guide/*.mdchanges/