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

supporting carriage return #194

Merged
merged 2 commits into from
Jun 19, 2015
Merged

supporting carriage return #194

merged 2 commits into from
Jun 19, 2015

Conversation

flyfy1
Copy link
Contributor

@flyfy1 flyfy1 commented Jun 18, 2015

Carriage return is used in Mac OS.

Currently, if the .env file is like that (\r is the newline char in Mac):

api_key=foo\r
another_token=bar

then it would be parsed as: api_key=foo\ranother_token=bar

@bkeepers
Copy link
Owner

@flyfy1 thanks for the pull request! It looks like there are a couple style issues causing the build to fail. You can see the failures locally by running bundle exec rubocop.

@flyfy1
Copy link
Contributor Author

flyfy1 commented Jun 19, 2015

@bkeepers Thx for the comments, fixed :)

bkeepers added a commit that referenced this pull request Jun 19, 2015
supporting carriage return
@bkeepers bkeepers merged commit d2e4549 into bkeepers:master Jun 19, 2015
sheax0r pushed a commit to sheax0r/dotenv that referenced this pull request Sep 13, 2015
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