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(validate_rd_policy) expand relative path for require #201

Merged
merged 2 commits into from
Mar 11, 2016
Merged

fix(validate_rd_policy) expand relative path for require #201

merged 2 commits into from
Mar 11, 2016

Conversation

igalic
Copy link
Contributor

@igalic igalic commented Mar 9, 2016

this should address #125

@jyaworski
Copy link
Member

Just go ahead and pin rake as part of this PR. Or rebase, because it may already be done.

@igalic
Copy link
Contributor Author

igalic commented Mar 10, 2016

oh come on now

manifests/config/global/ssl.pp:24:140chars:WARNING:line has more than 140 characters
manifests/config/resource_source.pp:106:140chars:WARNING:line has more than 140 characters

@igalic
Copy link
Contributor Author

igalic commented Mar 10, 2016

added a fix

@jyaworski
Copy link
Member

Looks like we're still failing.

why does it need this many ../..?!!

anyway, this should address #125
bastelfreak added a commit that referenced this pull request Mar 11, 2016
fix(validate_rd_policy) expand relative path for require
@bastelfreak bastelfreak merged commit 647ea85 into voxpupuli:master Mar 11, 2016
@igalic igalic deleted the require branch March 11, 2016 14:04
@igalic igalic mentioned this pull request Mar 11, 2016
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.

3 participants