-
Notifications
You must be signed in to change notification settings - Fork 271
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
Fix a typo in hammer_root.yml.epp #1173
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tests fail because we also have the statement there:
# Credentials. You'll be asked for the interactively if you leave them blank here |
13515ce
to
c333d2b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like you missed one so the tests are still failing
c333d2b
to
9ef5b90
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I manually fixed it up so I can release it.
Thanks @ekohl and sorry for taking so long, I was going to take a look later this week. |
This fixes a small typo in
hammer_root.yml.epp
. (It was previously fixed in theforeman/hammer-cli-foreman@11c2bf5.)