Skip to content
This repository has been archived by the owner on Dec 8, 2023. It is now read-only.

Maybe we should allow the user to configure this by supplying a config object as well and not strictly enforce config through env vars? #2

Open
lxkuz opened this issue Mar 11, 2021 · 0 comments
Labels
enhancement New feature or request

Comments

@lxkuz
Copy link
Collaborator

lxkuz commented Mar 11, 2021

Maybe we should allow the user to configure this by supplying a config object as well and not strictly enforce config through env vars?

I'm thinking that if an object is supplied as the argument to the function, we should treat that as a client config object, if not - default to FromEnv. If we decide to go for this, it could be good if we also cover it with a unit test for each of the paths.

Originally posted by @simskij in #1 (comment)

@simskij simskij added the enhancement New feature or request label Apr 23, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants