Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Output Aliases when running mage -l #456

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

daegalus
Copy link

We use mage for a few projects, and noticed a minor discovery problem of the aliases we setup for our targets. While yes you can run mage -h target to see aliases, that's not an obvious command to know about.

I have added the output of Aliases and to what targetname they point to in the mage -l output. That way it is more easily discoverable.

If there are no aliases setup, the Aliases section won't show up. I used an actual go if block instead of as a template one as it made err variable management a bit simpler. But I can make adjustments if it is preferred.

No new tests were added as a bunch of existing tests would fail if Aliases always showed up, and the ImportLists test fails if it doesn't show up.

I can add additional tests if requested though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants