-
Notifications
You must be signed in to change notification settings - Fork 4
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
Document the version of Herocions this gem supports? #33
Comments
Thank you for creating the issue, @nickjj.
Yes, that'd be great. Would you mind creating a PR? Thanks.
At the moment we are syncing it manually, but I'm open to hearing ideas on how we can achieve this programmatically. |
Done.
How do you sync it manually? |
I often visit the heroicons official repo and scan their commits. If they add/update icons, I manually update this repo too. |
Is this the general workflow?
For step 4 do you grab the changes from the |
I add the icons to
I copy the icons from https://github.com/tailwindlabs/heroicons/tree/master/src. Yes, To summarize, this is the workflow:
|
I think these steps could be automated in a GitHub Action. A daily or weekly script could:
At this point you can merge it and cut a gem release. I don't know your workflow for updating the changelog but that step could be done manually in the PR or after you merge it but before you build a new gem. What do you think? |
This seems like the best option in my opinion. Yeah, I'm pretty happy with the steps that you described above. I think that would get rid of all the manual steps that I do right now to update the icons. One question: How do we test the workflow? Do we have to wait for a new release from https://github.com/tailwindlabs/heroicons to verify that the workflow works as expected? |
Do you want me to close this issue and make a new for the automation workflow?
For development I'd suggest creating it all as an offline script that you can run on the command line on your dev box without needing to push any commits. This way it can be tested in isolation with a number of different inputs. Then once it's good to go drop it into a GitHub Action. As a first-time-end-to-end test maybe a 2nd private repo could be created just to make sure the GitHub Action portion works. I don't really know of a better way to test GitHub Actions unless you start looking into solutions like https://github.com/nektos/act (I haven't yet btw). |
Sure, that would be great for keeping issues separate.
Got it. Looks like a cool project on its own to learn and explore more about GitHub actions. Thanks for the help. |
This issue's task of documenting the herocions version is complete so I closed it. A continuation of the discussion around making it easier to keep this gem up to date has been opened here: #36 |
Hi,
Great gem!
I'm about to try it out and was wondering if you were open to documenting the version of Herocions used in the readme file?
I know it mentions there's 292 icons which matches the 292 icons available in Heroicons v2.0.16 based on viewing https://heroicons.com/ at the time of making this issue (it may change in the future).
For folks who might be on the fence on trying this out, it would be beneficial to see something like:
On that note, are you programmatically syncing releases to Tailwindlabs' repo or is it done manually?
The text was updated successfully, but these errors were encountered: