Skip to content

Conversation

@yang1992
Copy link

@yang1992 yang1992 commented Jun 6, 2018

for example:
{"config": ["{"sub_config":xxx}"]}

viper.Get("config") returns [map[sub_config:xxx]],but we hope all return data is viper, so we can use viper function to get, not map or any other struct.

Each viper instance need returns sub viper intance list which can deep traverse data.

@CLAassistant
Copy link

CLAassistant commented Jun 6, 2018

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


yangheng seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@andig
Copy link
Contributor

andig commented Mar 8, 2020

@yang1992 your PR makes perfect sense and is exactly what I need in order to pre-populate objects and use viper.Unmarshal to apply additional settings.

Would you mind signing the CLA so this could get merged?

@ps23
Copy link

ps23 commented May 26, 2020

Can this be signed?

@andig
Copy link
Contributor

andig commented May 26, 2020

@ps23 #863 is a signed replacement.

@swengineer404
Copy link

Can we get an update on this please?

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.

5 participants