Skip to content

Mac-friendly changes #9

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Jul 31, 2014
Merged

Mac-friendly changes #9

merged 3 commits into from
Jul 31, 2014

Conversation

@steeef
Copy link
Contributor

@steeef steeef commented Jul 31, 2014

Had a couple problems running this from OS X:

  1. Escape characters weren't working, so I switched \e to \x1B.
  2. The regexp for yaml/yml wasn't working, so I simplified it. I also allowed it to check eyaml/eyml files, which are used with hiera-eyaml (https://github.com/TomPoulton/hiera-eyaml).

Let me know if you want these changes in separate requests.

drwahl added a commit that referenced this pull request Jul 31, 2014
@drwahl
Merge pull request #9 from steeef/master
7d12833 
Mac-friendly changes
@drwahl drwahl merged commit 7d12833 into drwahl:master Jul 31, 2014
@drwahl
Copy link
Owner

drwahl commented Jul 31, 2014

Thanks for the contribution! Maybe I need to pick up a MacBook in the near future...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@steeef @drwahl