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

(SDK-305) Answer file to cache module interview answers, template-url etc #132

Merged
merged 5 commits into from
Jul 13, 2017

Conversation

rodjek
Copy link
Contributor

@rodjek rodjek commented Jul 4, 2017

Adds the PDK::AnswerFile class to handle reading and saving user-supplied answers to questions.

The responses from the module interview for the following questions are saved and used as the default values for new modules:

  • Forge username
  • Author name
  • License

If the license is specified on the command line with --license, then that value is saved to the answer file as well.

Additionally, if the user specifies their own module template using --template-url, that value is also saved and used automatically for any subsequent modules.

I had to add a hidden option to the CLI (--answer-file) in order to run the acceptance tests with clean answer files. Not sure if we want to make this option visible or not (my gut reaction is to keep it hidden unless there is a need for it).

@rodjek rodjek force-pushed the answer_file branch 8 times, most recently from c4dd18c to 97a1514 Compare July 6, 2017 09:38
@scotje
Copy link
Contributor

scotje commented Jul 7, 2017

This still WIP @rodjek ?

@rodjek rodjek force-pushed the answer_file branch 3 times, most recently from 30493b1 to 880856b Compare July 10, 2017 03:50
@rodjek rodjek changed the title [WIP] Answer file to cache module interview answers, template-url etc (SDK-305) Answer file to cache module interview answers, template-url etc Jul 10, 2017
@rodjek rodjek force-pushed the answer_file branch 3 times, most recently from d0cfc95 to 6e8780e Compare July 10, 2017 06:24
@rodjek
Copy link
Contributor Author

rodjek commented Jul 10, 2017

@scotje not anymore :)

@scotje
Copy link
Contributor

scotje commented Jul 10, 2017

Hahah, OK I'll take a look tomorrow then. :)

@bmjen bmjen self-requested a review July 13, 2017 16:50
@bmjen bmjen merged commit 0621158 into puppetlabs:master Jul 13, 2017
@bmjen bmjen added the feature label Jul 13, 2017
@rodjek rodjek deleted the answer_file branch July 18, 2017 22:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants