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.

Sign up for GitHub

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

Jump to bottom

✨ `[config]` Enable `time.Time` as valid configuration value #280

Closed

acabarbaye wants to merge 1 commit into master from handle-date

Contributor

acabarbaye commented Jun 26, 2023

Description

viper does not allow configuration fields to be time.Time. This PR is to enable this.

Test Coverage

This change is covered by existing or additional automated tests.
Manual testing has been performed (and evidence provided) as automated testing was not feasible.
Additional tests are not required for this change (e.g. documentation update).


          ✨ [config] Enable time.Time as valid configuration value

3f5d311

acabarbaye closed this

Contributor Author

acabarbaye commented Jun 28, 2023

limitation in mapstructure mitchellh/mapstructure#334

acabarbaye deleted the handle-date branch

June 28, 2023 15:56

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet