-
Notifications
You must be signed in to change notification settings - Fork 88
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
(CAT-1869) - Add puppet-modulebuilder to pdk bundled gems #869
Conversation
be1c5a3
to
3b0cec7
Compare
089f7f5
to
0fd6b2e
Compare
0fd6b2e
to
fb2676a
Compare
This commit adds the puppet-modulebuilder gem to the gems bundled with the pdk. This gem is no required for the module build functionality.
fb2676a
to
19f43c7
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.
LGTM
Sorry forgot one thing, could you make sure you can build the pdk runtime on all supported platforms using Vanagon Generic Builder before merging? The reason is each commit to the runtime causes all runtimes to be rebuilt |
Thanks @joshcooper! Going off the other PRs to this repo, here's my best attempt at that.. |
This PR adds the puppet-modulebuilder gem to the gems bundled with the pdk. This gem is now required for the module build functionality.