depends on knowing what the tools are => #191

Josh to find tools, Elango to help add them to CI.

Example: https://github.com/unicode-org/unicodetools/blob/main/docs/idna.md
--> add an entry in the issue description/checklist above for GenerateIdna.java
--> also add an entry for GenerateIdnaTest.java

Hi, just checking -- any progress? I think "Josh to find tools" comes first...?

@markusicu @srl295 I went through and noted what appeared to be Tool-like items and added to the checklist above. Some of them might not be so easy (or appropriate) to auto-run.

Thanks @josh-hadley ! Taking a look...

• EmojiFrequency: We don't seem to have its input file vendorRaw.tsv in the repo. Provide a dummy file?
• ToolUnicodePropertySource: This is not a tool. No main(), and it's a class that provides properties to the tools, rather than being a tool.
• GenerateEnums: This tool wants to overwrite two .java files in the repo source. For CI, we could add an option to have it write to /tmp or similar. Would be useful to fail if the generated output differs from the repo files.
• GenerateUnihanCollatorFiles: This is a tiny tool that inserts the output data from GenerateUnihanCollators into CLDR data files. Probably ok not to add to CI.

@echeran do you feel unblocked?

@markusicu yes, that should unblock me

• EmojiFrequency.java -- using non-public files from vendors, use stub files for smoke testing
• GenerateEmoji.java needs a path to a separate, private "images" repo -- maybe use a workflow that is only invoked manually; could use a "deploy key" (GitHub secrets) on the images repo to ssh-fetch the images repo -- agreed out of scope for this issue, skip
• UCA.Main -- run on data/dev files

@echeran EmojiFrequency.java still TBD?

@echeran EmojiFrequency.java still TBD?

@markusicu Yep, still need a dummy .tsv file from @macchiati for running that in CI.

@echeran EmojiFrequency.java still TBD?

@markusicu Yep, still need a dummy .tsv file from @macchiati for running that in CI.

@macchiati did you provide a dummy .tsv file to @echeran for running this tool?

@echeran EmojiFrequency.java still TBD?

@markusicu Yep, still need a dummy .tsv file from @macchiati for running that in CI.

@macchiati did you provide a dummy .tsv file to @echeran for running this tool?

ping @macchiati

I looked at this, and it would take some work. So I would put it at a low priority.

Creating a dummy .tsv file would take some work? How much? An hour? An afternoon? A couple of days?
The goal is to routinely run EmojiFrequency.java so that we catch if & when it gets broken, not next time we need to run it.

Closing this issue as done. The one remaining tool (EmojiFrequency.java) will be tracked via new issue #338.