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

Convert to puppet 4/5 data types #58

Merged
merged 1 commit into from
Jun 30, 2017
Merged

Convert to puppet 4/5 data types #58

merged 1 commit into from
Jun 30, 2017

Conversation

matonb
Copy link
Contributor

@matonb matonb commented Jun 29, 2017

RFC

Replace validate_x functions with Puppet data types

Rename 'uncaught' tests

  • spec/defines/icp_access.rb
  • spec/defines/sslproxy_cert_error.rb

Fix (?) extra_config test, order parameter specified as integer instead of string.

@matonb matonb changed the title WIP: Convert to puppet 4/5 data types RFC: Convert to puppet 4/5 data types Jun 29, 2017
@traylenator traylenator self-requested a review June 30, 2017 08:08
Copy link
Contributor

@traylenator traylenator left a comment

Choose a reason for hiding this comment

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

Thanks for doing this, it's time for sure.

@matonb
Copy link
Contributor Author

matonb commented Jun 30, 2017

@traylenator No problem, like you say it's definitely time to get in to the habbit with puppet 5 around the corner

@matonb
Copy link
Contributor Author

matonb commented Jun 30, 2017

@traylenator Do you want the commits squashed or rebased ?

@traylenator
Copy link
Contributor

Please rebase, squash.

@traylenator
Copy link
Contributor

traylenator commented Jun 30, 2017

Need to update metadata for puppet requirements.

Already done in fact.

Rename 'uncaught' tests
* spec/defines/icp_access.rb
* spec/defines/sslproxy_cert_error.rb

Fix (?) extra_config test, order specified as integer instead of string - RFC
@matonb
Copy link
Contributor Author

matonb commented Jun 30, 2017

@traylenator rebased as requested

@traylenator traylenator merged commit e3a09f2 into voxpupuli:master Jun 30, 2017
@matonb matonb deleted the data_types branch June 30, 2017 11:21
@alexjfisher alexjfisher changed the title RFC: Convert to puppet 4/5 data types Convert to puppet 4/5 data types Jul 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants