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

Removed logical branch (#2355) #2364

Merged
merged 1 commit into from
Oct 13, 2019

Conversation

houtd
Copy link
Contributor

@houtd houtd commented Oct 13, 2019

Fixes: #2355
Removed logical branch which was previously required when appending scenario name to instances.

Appending scenario name to instances was removed in #896

Source for information:
1dca01c#diff-fd0ed9e86ebd26cf708da3d42fbf2eacL582

PR Type

  • Bugfix Pull Request

…cenario name to instances.

Appending scenario name to instances was removed in ansible#896

Signed-off-by: houtd <56220513+houtd@users.noreply.github.com>
@houtd houtd force-pushed the fix-identical-logical-branches branch from 79311de to f4ac7d0 Compare October 13, 2019 01:26
Copy link
Contributor

@decentral1se decentral1se left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this good detective work!

🚀

@decentral1se decentral1se merged commit 233ff27 into ansible:master Oct 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Identical code for different logical branches in molecule\provisioner\ansible.py
3 participants