-
-
Notifications
You must be signed in to change notification settings - Fork 144
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
Puppet 4.0 support #106
Puppet 4.0 support #106
Conversation
Conflicts: metadata.json
# Conflicts: # metadata.json
@@ -26,11 +26,11 @@ | |||
"requirements": [ | |||
{ | |||
"name": "pe", | |||
"version_requirement": ">=3.7.0 <4.0.0" | |||
"version_requirement": ">=3.7.0 <4.3.0" |
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.
4.3.0 was just released, is there a particular reason to stay below that version when testing?
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.
pretty sure we can do <5.0.0 here
I don't have any technical comments but did add a few syntax comments. Also see #96 (comment), same failures are occuring here. |
Signed-off-by: Andrew Grimberg <tykeal@bardicgrove.org>
FIX linting
The format parameter needs to not have a leading . or all of the spec tests need to be fixed to pass it in. As removing it is the simpler option this does that. Secondly, the calculated file names for the deploy need to have the file extension . added. Also, the calculated file name for pre-7.0.0 had been destroyed, this puts it back to what it should be. Signed-off-by: Andrew Grimberg <tykeal@bardicgrove.org>
Fix deployment files to unblock spec
Hello: What's the status of this PR? |
@ffquintella can we convince you to rebase & squash this pr? |
Igalic. For me it's ok. I will close de PR. And as soon as I have some extra time I will watch into rebasing |
This a new version with some minor bugfixes