feat: alias management commands (#25)

[roele]

This is a draft on the alias management commands.

Couple of caveats

• not sure if 2-3 arguments should be used rather than the proposed dot notation "<PLUGIN>.<ALIAS>" (or even "<PLUGIN@ALIAS>"?)
• ls command uses option -p to specify a plugin but since this is for filtering this might not be an inconsistency
• adding an alias adds an empty [alias] section in the config.toml
• found a discrepancy with aliases (singular/plural) in rtxrc.rs

[jdx]

these shouldn't use pub (unless I'm missing something)

[roele]

i was wondering about this as well but it seems other commands also define those as public

[jdx]

not sure if 2-3 arguments should be used rather than the proposed dot notation "." (or even "PLUGIN@ALIAS"?)

I think I like multiple arguments the most. For runtimes I've settled on using plugin@version but that's so we can use multiple, e.g.: rtx i nodejs@10 nodejs@12. That seems less useful here.

ls command uses option -p to specify a plugin but since this is for filtering this might not be an inconsistency

This is because it's a root command. It has to use an option because if you ran rtx alias get it wouldn't be possible to know if "get" was the name of a plugin or the name of a subcommand.

adding an alias adds an empty [alias] section in the config.toml

worth noting but I think we can live with this

found a discrepancy with aliases (singular/plural) in rtxrc.rs

yep, good catch

[codecov]

Codecov Report

Base: 81.77% // Head: 82.14% // Increases project coverage by +0.37% 🎉

Coverage data is based on head (ae06e76) compared to base (934271e).
Patch coverage: 100.00% of modified lines in pull request are covered.

❗ Current head ae06e76 differs from pull request most recent head d71d2d8. Consider uploading reports for the commit d71d2d8 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #159      +/-   ##
==========================================
+ Coverage   81.77%   82.14%   +0.37%     
==========================================
  Files          86       89       +3     
  Lines        2809     2845      +36     
==========================================
+ Hits         2297     2337      +40     
+ Misses        512      508       -4     

Impacted Files	Coverage Δ
src/cli/alias/get.rs	100.00% <100.00%> (ø)
src/cli/alias/mod.rs	100.00% <100.00%> (ø)
src/cli/alias/set.rs	100.00% <100.00%> (ø)
src/cli/alias/unset.rs	100.00% <100.00%> (ø)
src/config/config_file/rtxrc.rs	79.02% <100.00%> (+1.42%)	⬆️
src/cli/self_update.rs	0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[jdx]

this is probably out of scope, but I wonder if we could generate this path using env::CONFIG_FILE_PATH or whatever that is.

Don't worry about this one unless you're particularly curious.

[jdx]

I'll go ahead and merge this once the tests pass

[jdx]

thanks again for the help!