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

Pin puppet-lint to 0.3.2 #210

Merged
merged 1 commit into from
Aug 19, 2014
Merged

Pin puppet-lint to 0.3.2 #210

merged 1 commit into from
Aug 19, 2014

Conversation

cmurphy
Copy link
Contributor

@cmurphy cmurphy commented Aug 19, 2014

No description provided.

@jhoblitt
Copy link
Contributor

Why does the version need to be pinned?

@cmurphy
Copy link
Contributor Author

cmurphy commented Aug 19, 2014

The new release of puppet-lint causes all of the modules to break. This module would have failed travis checks in #209 if they had been allowed to complete. We're pinning puppet-lint across all the supported modules until the changes to puppet-lint have been worked out.

@jhoblitt
Copy link
Contributor

@cmurphy I hadn't hit that yet. Sounds reasonable. I'm going to go ahead and merge it before travis finishes (or even starts) since it'll likely fail anyways.

jhoblitt pushed a commit that referenced this pull request Aug 19, 2014
Pin puppet-lint to 0.3.2
@jhoblitt jhoblitt merged commit feba309 into puppetlabs:master Aug 19, 2014
@cmurphy
Copy link
Contributor Author

cmurphy commented Aug 19, 2014

Well the point of this commit is to prevent it from failing :)

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.

3 participants