-
Notifications
You must be signed in to change notification settings - Fork 31
Fix badges #406
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
Fix badges #406
Conversation
|
Going to add a little update for this, just yesterday noticed that I forgot chests and CNC. |
|
Added mtt badges there too while at it, @BuckarooBanzay last commit can be dropped if you think these should not be there. |
This reverts commit 106a9dd.
Mineunit failed regression tests, click for detailsRegression test log for Technic CNC:Regression test log for Technic Chests:Regression test log for Technic: |
|
Reverted mtt, my best understanding after going through things around it was that it doesn't really do much with CNC or chests. |
Badges for mtt, luacheck and mineunit in README.md are showing status from latest run without taking branch into account.
Statuses should only come from main release branch which is master (or default where it can be used).
Also linked badges in a way that makes it easy to find actual latest report for the master branch.
Report for most current workflow execution, where badge actually comes from, will be the very first entry instead of being hidden somewhere below development/bugfix/pending pull requests/etc.