alerque
commented
3 years ago Actually the situation isn't as bad as I thought.
- The PR and 2 workflows it introduced actually worked fine when they were merged.
- The lint workflow still works, but is throwing errors because a later merge introduced lint warnings (#646).
- The test runner is also working, just the badge is broken.
I'll submit a fix for the badge. I suggest now that the CI actions are working that master be marked as a protected branch and the linter and at least some of the spec test runs be made into required checks to merge PRs.
After merger of #645 everything should have been green, but since I could only test on my fork rather than this repo it was a bit hard to test. Now with the GH Actions workflow at least existing in the repo it can be run locally and PRs should reflect the status. I'll look into what I missed—probably somewhere I left in a path to my fork or something dumb like that.
Feel free to assign this to me in case I don't get to it tonight and forget about it tomorrow ;-)