chshersh / tool-sync

🧰 Download pre-built binaries of all your favourite tools with a single command
https://crates.io/crates/tool-sync
Mozilla Public License 2.0
69 stars 16 forks source link

Outputs tools URLs in the default-config command #123

Closed crudiedo closed 1 year ago

crudiedo commented 1 year ago

Resolves #111

This PR adds a new function "fmt_tool_names_info" that accepts a closure with both name and tool info of every record.

The config_template function calls it to output the tool URL along with applying the alignment based on the longest name length.

MitchellBerend commented 1 year ago

Also don't forget to add your change to the CHANGELOG.md so this pr isn't missed on the next release by accident!

crudiedo commented 1 year ago

@MitchellBerend thank you for your review and help - that's my first pull request, so it got a bit hasty (:

As suggested, I've updated the install command error output to display the repo URL as well.

I've also removed the initial fmt_tool_names function - I understand that's a bit risky to do, but as I can see it's not being used anymore.

I really appreciate your help.