这是indexloc提供的服务，不要输入任何密码
Skip to content

Bumping dotenv version #12

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
Closed
wants to merge 8 commits into from
Closed

Bumping dotenv version #12

wants to merge 8 commits into from

Conversation

@groovenectar
Copy link

@groovenectar groovenectar commented Dec 3, 2018

The newer version of dotenv "~> 2.5" fixes syntax issues.

This is valid for a .env file but breaks in dotenv "~> 0.9":

DIRECTORIES=(
	"${PATH}"
	"${PATH}/path123"
	"${PATH2}/other/path"
)

@groovenectar groovenectar mentioned this pull request Dec 3, 2018
Open
@groovenectar
Copy link
Author

groovenectar commented Dec 5, 2018

This is also valid Bash:

declare -A ARRAYVAR

The current version of vagrant-env will throw this error:

define "declare -A ARRAYVAR" doesn't match format (Dotenv::FormatError)

Using dotenv "~> 2.5" fixes this as well.

Any reasons not to use the latest version to more fully support Bash syntax?

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.

1 participant

@groovenectar