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

Fix puppet strings warnings and minor README.md update #191

Merged
merged 2 commits into from
Jan 31, 2017
Merged

Fix puppet strings warnings and minor README.md update #191

merged 2 commits into from
Jan 31, 2017

Conversation

vinzent
Copy link
Contributor

@vinzent vinzent commented Jan 31, 2017

Puppet 3.8.7 was specified in README.md but this is now Puppet 4 only.

README.md Outdated
* run acceptance tests:

```
BEAKER_debug=yes BEAKER_set="centos-6-x64" PUPPET_INSTALL_TYPE="agent" be rake beaker
Copy link
Member

Choose a reason for hiding this comment

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

not everybody uses this alias. should we use bundle exec instead?

@vinzent
Copy link
Contributor Author

vinzent commented Jan 31, 2017

added the centos 6 and 7 nodesets too. PR open in modulesync_config: voxpupuli/modulesync_config#313

@vinzent
Copy link
Contributor Author

vinzent commented Jan 31, 2017

don't merge.

@vinzent vinzent closed this Jan 31, 2017
@vinzent
Copy link
Contributor Author

vinzent commented Jan 31, 2017

sorry, I thought the docs/ folder was messed up. but it was only my try to run puppet strings server which doesn't do what i've expected.

@vinzent vinzent reopened this Jan 31, 2017
@bastelfreak bastelfreak merged commit f3a86c6 into voxpupuli:master Jan 31, 2017
@vinzent vinzent deleted the update_docs branch February 3, 2017 21:31
EmRowlands pushed a commit to EmRowlands/puppet-selinux that referenced this pull request Mar 29, 2023
Fix puppet strings warnings and minor README.md update
cegeka-jenkins pushed a commit to cegeka/puppet-selinux that referenced this pull request Jan 10, 2025
Fix puppet strings warnings and minor README.md update
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants