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

feat: add support for devcontainer up --workspace-env-file #978

Merged
merged 2 commits into from
Apr 2, 2024

Conversation

eduardodbr
Copy link
Contributor

Fixes #827

Each file from the list is parsed for key value pairs in the format key=value and merged with the remaining --workspace-env variables.

Signed-off-by: eduardodbr <eduardodbr@hotmail.com>
cmd/up.go Outdated Show resolved Hide resolved
cmd/up.go Outdated Show resolved Hide resolved
e2e/tests/up/up.go Show resolved Hide resolved
@89luca89
Copy link
Member

Thanks a lot @eduardodbr for the PR!

Just a couple of adjustments 😄 great work, thanks!

Signed-off-by: eduardodbr <eduardodbr@hotmail.com>
@eduardodbr
Copy link
Contributor Author

Thanks a lot @eduardodbr for the PR!

Just a couple of adjustments 😄 great work, thanks!

Done the suggested changes, thanks. Also changed workspace-env-file to be StringSliceVar instead of StringArrayVar, this way it already parses the arguments as comma-separated list of filenames

@89luca89
Copy link
Member

89luca89 commented Apr 2, 2024

Thanks a lot @eduardodbr LGTM, running the test suite, and if all is green will merge 👍

@89luca89 89luca89 merged commit 1d5f761 into loft-sh:main Apr 2, 2024
24 checks passed
@89luca89
Copy link
Member

89luca89 commented Apr 2, 2024

Thanks a lot!

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.

Add support for devcontainer up --secrets-file
2 participants