cross icon indicating copy to clipboard operation
cross copied to clipboard

change target info to json and display them neatly

Open Emilgardis opened this issue 2 years ago • 2 comments

this is not a serious pr, just an attempt that maybe can become something

Emilgardis avatar Jun 18 '22 08:06 Emilgardis

I think this is a good idea, since we can always generate the table on the Rust side from the JSON output, which is what this already does, and we can plug it into CI or something if needed in the future (maybe a good idea if we're planning a release, to ensure the README is correct). Do we need rayon though? It's a very heavy dependency (a good one, but might increase compile times by a lot).

Alexhuszagh avatar Jun 18 '22 15:06 Alexhuszagh

Do we need rayon though? It's a very heavy dependency (a good one, but might increase compile times by a lot).

Yeah, I placed it behind a feature to try mitigate that cost, but I thing it's impossible to not build it since we use --all-features

Used rayon because it took a long time to execute everything on my machine, I guess there is a way to do it without rayon but this was the simplest thing, can definitely remove it

Emilgardis avatar Jun 18 '22 15:06 Emilgardis