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

Add support for templating in vars_prompt #9789

Closed
wants to merge 1 commit into from

Conversation

nathancahill
Copy link
Contributor

No description provided.

@nathancahill nathancahill changed the title Add support for templating in vars_prompt - fixes #7959 Add support for templating in vars_prompt Dec 12, 2014
@sudarkoff
Copy link
Contributor

👍 Thanks, @nathancahill, I was just about to do the same.

I'd add a few tests for this.

@jeromedoyle
Copy link

Any update on this or alternate method to accomplish the same effect?

@bcoca
Copy link
Member

bcoca commented Jun 11, 2015

this probably needs to be redone as the whole system has drastically changed.

@bcoca bcoca added the needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. label Jun 11, 2015
@amenonsen
Copy link
Contributor

As @bcoca says, the code in devel is very different now, and achieving the object of this PR (which is slightly unclear; some examples/tests would be welcome) would need a rewrite. I don't think there's any point keeping this PR open.

@jimi-c jimi-c removed the P4 label Dec 7, 2015
@bcoca
Copy link
Member

bcoca commented Feb 2, 2016

closing this as current 2.0 should fix #7959 (as mentioned in that ticket).

@bcoca bcoca closed this Feb 2, 2016
@ansibot ansibot added bug This issue/PR relates to a bug. and removed bugfix_pull_request labels Mar 5, 2018
@ansible ansible locked and limited conversation to collaborators Apr 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug This issue/PR relates to a bug. needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants