-
Notifications
You must be signed in to change notification settings - Fork 11
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
feat: add create command #31
base: main
Are you sure you want to change the base?
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. |
Some things I want to do: