search

technomancy / slamhound

Slamhound rips your namespace form apart and reconstructs it.
Other
473 stars 38 forks source link

The alias thing is a bit of a pain #62

Closed phillord closed 10 years ago

phillord commented 10 years ago

It would be nice not to have to define an alias in my profile.clj. Installing as a plugin, then typing "lein slamhound" should work, I think!

technomancy commented 10 years ago

I don't understand; why is one line under :plugins better than one line under :aliases?

phillord commented 10 years ago

Well, you have to install slamhound by adding to your dependencies, and then adding an alias. Generally, when I am installing things that I run from leiningen, I add a single entity to the plugins.

It's not a major disaster, just a small improvement suggestion.

phillord commented 10 years ago

Looking through the closed issues, #57 is caused by the same (mistaken) assumption.

technomancy commented 10 years ago

Adding this via a plugin instead of a dependency makes things a bit more complicated because now you can't invoke it from a repl or from your editor. So for the recommended usage (invocation from the CLI is too slow for most people) you still need to add it in two places. Also, it hides the version number of the actual dependency inside the plugin source, which can be confusing especially when it comes to higher-order tasks.

guns commented 10 years ago

Hello,

I am going to close this since this is a feature, not a bug. Thank you!