-
Notifications
You must be signed in to change notification settings - Fork 0
feat: #6 support environment variables #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.
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
Conversation
Olwiba
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The code looks good to me, I will leave an approval for someone else to get a second set of eyes :)
phillip-haydon
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🌯
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Woops forgot to change Comment > Approve :)
Edit: Oh it needs @TheRealAgentK to approve too.
TheRealAgentK
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
feat: #6 support environment variables
Description 📝
Type of change
Updates
config_props.dartwhich loads common configuration parameters.environment.dartwhich allows to override Environment variables for testing.config_props_test.dart.symbolsandsourcemapto use the config props class.Screenshots 📷
e.g. calling to
symbols listwith env variables:(
--verboseprints the config params)Test plan 🧪
Tested manually and created unit test
Author to check 👓
Reviewer to check ✔️