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

idem minor version number cleanup #3

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

Conversation

TheBigBear
Copy link

those version numbers look like salt version numbers and not idem ones. The last bit about the output of 'idem exec test.version' is invented. It blows up on my site so I do not know what this would actually report? I assume it might be the output in invented, but maybe that is well off.

those version numbers look like salt version numbers and not idem ones. The last bit about the output of 'idem exec test.version' is invented. It blows up on my site so I do not know what this would actually report? I assume it might be the output in invented, but maybe that is well off.
@TheBigBear
Copy link
Author

@nicholasmhughes ok, I am pretty sure the last bit telling user to do idem exec test.version was simply a copy and paste error from saltenv to idemenv, right?

So I suggest to simply close with one or more of the following tests in closing:
1 idem --version
2 idem exec test.ping
3 idem exec test.more testing test=False account_id=$(whoami)

@nicholasmhughes
Copy link
Member

Sounds good! Let's go with option 2. Thanks!

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.

2 participants