Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@hazzik
Copy link

@hazzik hazzik commented Mar 10, 2025

No description provided.

@hazzik hazzik requested review from a team as code owners March 10, 2025 05:57
@hazzik hazzik requested a review from rboyer March 10, 2025 05:57
@hazzik hazzik changed the title chore: replace github.com/mitchellh/mapstructure with github.com/go-v… chore: replace github.com/mitchellh/mapstructure with github.com/go-viper/mapstructure/v2 Mar 10, 2025
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Thanks for raising the PR but not exactly sure why we need this change. Were you getting blocked on anything due to the original dependency ?

@hazzik
Copy link
Author

hazzik commented Mar 10, 2025

Not blocked. Just do not like having outdated dependencies. Original dependency was deprecated by the owner, and the new one is a "blessed fork" that will be supported by viper team. And serf seems to be a low level dependency in quite a lot of packages that we use. So my motivation was to have the train moved in that direction.


require (
github.com/armon/circbuf v0.0.0-20150827004946-bbbad097214e
github.com/go-viper/mapstructure/v2 v2.2.1

Choose a reason for hiding this comment

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

This can probably updated to v2.4.0 now; https://github.com/go-viper/mapstructure/releases/tag/v2.4.0

@thaJeztah
Copy link

cc @rmainwork are you a maintainer for this repo? Was curious as well if this project would update to the maintained / blessed fork (I'm trying to prevent dependency sprawl in my projects)

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.

2 participants