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

Support multiline values #233

Merged
merged 1 commit into from Feb 28, 2024
Merged

Support multiline values #233

merged 1 commit into from Feb 28, 2024

Conversation

gonzaloriestra
Copy link
Contributor

Adds support for multiline values:

echo -e "a=\"1\n2\"\nb=3" | npm run our:bin env2json
  {"a":"1\n2","b":"3"}

echo '{"a":"1\\n2","b":3}' | npm run our:bin json2env
  a="1\n2"
  b=3

@gonzaloriestra
Copy link
Contributor Author

@balupton could you please review this? Thanks!

@balupton
Copy link
Member

Tests failed on Windows. Can you fix or reproduce that locally?

@balupton
Copy link
Member

balupton commented Feb 28, 2024

Actually seems the windows failure is prior to this PR
https://github.com/bevry/envfile/actions/runs/7360505711

will merge so it gets published to the @next npm tag, and then will look into windows and publishing to latest either this week or next week

if you can fix the windows tests, in a subsequent PR that would be awesome!

@balupton balupton merged commit dc6083d into bevry:master Feb 28, 2024
10 of 14 checks passed
@balupton
Copy link
Member

Actually, leave the windows issue with me. I believe I know what it is.

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.

None yet

2 participants