Fix CI status badge in README
j4mie opened this issue · comments
This seems to just be showing the status of whatever the most recent build happened to be, rather than the status of the main
branch. Need to investigate. Thanks for the report @simkimsia
@j4mie if you are open to it, I recommend use precommit to replace variables in order to replace {{ current.branch }} with the correct branch name regardless
See https://stackoverflow.com/a/67919523/80353
and then put ?branch={{ current.branch }}
as per https://docs.github.com/en/actions/monitoring-and-troubleshooting-workflows/adding-a-workflow-status-badge#using-the-branch-parameter
What say you? I am happy to help by writing a PR. though i have never used pre-commit before. This is my way of helping. I ❤ django-readers
Thanks! Your "quick fix" is perfect. I'm not a big fan of pre-commit personally.
Fixed in #63
Ah this still doesn't work 😆 CI doesn't run on main
so it just says "no status". What we want is the status of whatever the last thing that was merged into main is.. which probably isn't possible?
I'm not actually sure what value this badge really adds... maybe we should just remove it.
I think a good way to do this would be to add
on:
push:
branches:
- main
To the GitHub Actions config so that the tests do run against main
after a merge. This means the badge represents "do the tests pass against current main
, which I think is what we want.
sounds good. wanna try?
love the green color passing badge! 🙌