search

iknite / cookiecutter-ansible-role

Cookie cutter recipe to easily create ansible roles.
BSD 3-Clause "New" or "Revised" License
87 stars 31 forks source link

If/Else logic in post_gen_project.py #10

Open nicolepickle opened 5 years ago

nicolepickle commented 5 years ago

Thank you for this repo, it's a terrific time saver. One issue I noticed:

if read_user_yes_no(folder['question'], default_value=u'no'): if read_user_yes_no(folder['pre_hint'], default_value=u'no'):

Should be

if read_user_yes_no(folder['question'], default_value=u'no') == 'yes': if read_user_yes_no(folder['pre_hint'], default_value=u'no') == 'yes':

Otherwise they always evaluate to true and we are still prompted for the values if we set the default to 'no'.

iknite commented 5 years ago

I'll look into it since it is an inverted dependency so maybe they change the functionality. I'll assume It's just me making bugs.

thanks!! I'm really grateful that it helps