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

(maint) Allow pdk bundle to work without -- #466

Merged
merged 1 commit into from
Apr 12, 2018

Conversation

DavidS
Copy link
Contributor

@DavidS DavidS commented Apr 4, 2018

By upgrading to the new version of CRI, the skip_option_parsing
call can be used to avoid the previous confusion of bundle vs pdk
options.

By upgrading to the new version of CRI, the `skip_option_parsing`
call can be used to avoid the previous confusion of bundle vs pdk
options.
@DavidS
Copy link
Contributor Author

DavidS commented Apr 4, 2018

This is a prerequisite for #331

@coveralls
Copy link

Coverage Status

Coverage increased (+0.04%) to 92.373% when pulling a50e844 on DavidS:skip-option-parsing into 83969ba on puppetlabs:master.

@@ -22,7 +22,7 @@ Gem::Specification.new do |spec|
spec.required_ruby_version = '>= 2.1.9'

spec.add_runtime_dependency 'bundler', '~> 1.15'
spec.add_runtime_dependency 'cri', '~> 2.9.1'
spec.add_runtime_dependency 'cri', '~> 2.10.1'
Copy link
Contributor

@bmjen bmjen Apr 12, 2018

Choose a reason for hiding this comment

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

Is this update required for the bundle command changes? I'd prefer if this was taken out and a new ticket created to update CRI, so we can update the vanagon component at the same time.

@bmjen bmjen merged commit 2c862ce into puppetlabs:master Apr 12, 2018
@DavidS DavidS deleted the skip-option-parsing branch April 23, 2018 15:31
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