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: env contains list of strings #255

Merged
merged 1 commit into from
Mar 2, 2022

Conversation

simon-an
Copy link
Contributor

@simon-an simon-an commented Dec 3, 2021

Signed-off-by: simon-an [email protected]

fixes #117

Copy link
Member

@matthiasbeyer matthiasbeyer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Codewise this looks good, but please fix the commit message and use a real signoff line! 😃

@simon-an
Copy link
Contributor Author

simon-an commented Dec 6, 2021

Ups. 😄 Done.

@simon-an
Copy link
Contributor Author

simon-an commented Dec 6, 2021

I noticed, that with my first implementation one have to decide what to use. Strings or lists of strings. So i added an option which allows adding the keys of environment variables to be listed for parsing.

@simon-an
Copy link
Contributor Author

simon-an commented Jan 3, 2022

Any chance we get this merged?
If you want to have a more sophisticated approach (instead of adding keys the parser should use), tell me in what direction to go and i will implement it.

@simon-an
Copy link
Contributor Author

@matthiasbeyer Hi, can we merge this now? checks are green and branch is up to date.

@matthiasbeyer
Copy link
Member

Yeah I think so. Please rebase the branch so the merge commits vanish! 👍

src/env.rs Show resolved Hide resolved
@matthiasbeyer matthiasbeyer merged commit 2e9ccf7 into rust-cli:master Mar 2, 2022
@simon-an simon-an deleted the parse-environment-list branch April 4, 2022 10:20
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.

environment variables list?
2 participants