Generate Markdown table of tools
Closes https://github.com/taiki-e/install-action/issues/440
This is still a bit of a mess, and the new logic is in the wrong spot - it is currently designed to easily update the existing manifests rather than supporting creating new manifests.
Key question atm is whether it is OK to add new fields website & license_markdown to both the base info and manifest structs.
Another is whether it is OK to create a lib.rs ? I moved everything to the lib.rs , but would be happy to move the unmodified parts back to main.rs to reduce the size of the change.
Thanks!
Key question atm is whether it is OK to add new fields
website&license_markdownto both the base info and manifest structs.
I'm okay with adding those as those are small enough information.
Another is whether it is OK to create a
lib.rs? I moved everything to thelib.rs, but would be happy to move the unmodified parts back tomain.rsto reduce the size of the change.
I'm okay with either.
Great.
Next two questions:
-
Is it ok to generate as
TOOLS.md, and link to it fromREADME.md? IMO the list is already getting long. I dont mind merging it into the README.md . It is only a bit more work to do that. -
Should the markdown generator be a separate tool , or run at the end of the existing
main.rs?
Is it ok to generate as
TOOLS.md, and link to it fromREADME.md? IMO the list is already getting long. I dont mind merging it into the README.md . It is only a bit more work to do that.
The current list is indeed long, so splitting it into separate files is okay.
Should the markdown generator be a separate tool , or run at the end of the existing
main.rs?
I think the latter is better at this time because it eliminates the need for changes to the CI script.
IIUC, the CI loops over each tool, in which case putting the markdown generation in the main.rs will mean it does that step many times - once per tool, instead of only once at the end. Anyways, I'll put it in main.rs, but put the logic in lib.rs, so it can become a separate bin later.
IIUC, the CI loops over each tool, in which case putting the markdown generation in the
main.rswill mean it does that step many times - once per tool, instead of only once at the end.
Oh, you are right. That is indeed inefficient and would be better to separate them.