Conversation
0f8a39c to
843aba5
Compare
|
The linter issue should be fixed in #28. |
|
@wesleytodd do you want me to update the README in this PR or in another one with a better structure? |
|
I know I already did this work, but I think it would be better to convert this into a GitHub action instead of a CLI |
I will take a strong stance on this. Every github action should be a light wrapper around a JavaScript library. And in the case of this one, generating a static site out of a status board is also a reasonable use case where a cli makese sense. It is common for people to author things like this in a very narrow way that limits it's use, and so I set this package up this way to avoid that fate. Nearly everything we do in OSS is best done as a small stand-alone library first which is wrapped in a UX second, and this is no exception. I will get on reviewing this code (might be tomorrow at this point), but I just wanted to reply on that point while it was on my mind. |
b1fa635 to
94b9fed
Compare
Some things I want to do: