README badges #169
README badges #169
Conversation
Also slightly reorder badges.
|
The GH Action fails because it checks the current readme, not the one after merging... a bit weird actually. I just used the suggested template at https://github.com/fair-software/howfairis-github-action, but maybe we should customize it to run on the actual checked out repository instead of on the current only main-branch version. |
There was a problem hiding this comment.
The branch the how-fair-is action is checking (main) can certainly lead to confusing situations later on. It would be worth our time to see if we can fix this to use the current branch instead. It is already complicated enough to understand what the actions are actually checking as it is.
Great that you were able to get the 5th check for how-fair-is! Great cleanup of repo and readme!
.github/workflows/fair-software.yml
Outdated
| env: | ||
| PYCHARM_HOSTED: "Trick colorama into displaying colored output" | ||
| with: | ||
| MY_REPO_URL: "https://github.com/${{ github.repository }}" |
There was a problem hiding this comment.
Doesn't seem super easy to make it work with our current branch instead of main. bummer.
There was a problem hiding this comment.
It was doable! However, for some reason howfairis doesn't properly detect the badge on the README.md... Shall we merge and see if that fixes it? If not, we should probably do a bug report with them.
Note: the badge that's mentioned ("see badge") is added to the top of the section in PR #169.
Fixes #128, i.e. it adds a green howfairis badge now that #113 has been merged, and adds a GH Action to check this from now on.
This PR also reorders badges a in three ways:
badges.mdis removed, further cleaning up the repo (again following Cleanup files #120).