-
Notifications
You must be signed in to change notification settings - Fork 72
Open
Description
While working #76, I had issues running all of the tests locally. Also, my PR didn't abide to go doc standards, which I also didn't catch. I think this repo could benefit from 2 things:
- Local CI Checks - This could be done using a
Makefile
, which would essentially be duplicating the checks locally so that a developer can verify their PR conforms to standards and the tests work. TheMakefile
could also automate the setup needed to run the tests locally. - Github Actions - I think the free account allows like 2k action minutes p/month before getting charged. I'm not sure what the owner is ok with, but maybe there could be a post check upon a merge to master. This would keep the usage minimal, even though it would be nice to have this run for each PR - easier said though when I'm not floating the bill 😄
Metadata
Metadata
Assignees
Labels
No labels