mitchellh / mapstructure

Go library for decoding generic map values into native Go structures and vice versa.
https://gist.github.com/mitchellh/90029601268e59a29e64e55bab1c5bdc
MIT License
7.93k stars 677 forks source link

Community should fork this project and maintain it on its own. #349

Open prochac opened 11 months ago

prochac commented 11 months ago

@mitchellh doesn't have time to maintain this project and plans to archive it.

https://github.com/mitchellh/mapstructure/issues/172

https://github.com/mitchellh/mapstructure/pull/328#issuecomment-1858922295

@sagikazarmark (Viper contributor, https://github.com/spf13/viper/graphs/contributors) shows interest to create fork under an umbrella of Viper project.

This issue was created mostly for better visibility and communication than in the linked PR.

mitchellh commented 11 months ago

Hi! Yes, I’m sorry for “breaking the news” in the comment. I always planned (and still do) to make a list of repositories I no longer wish to maintain and asking the community for a fork before archiving them. I wasn’t going to suddenly archive this repo, I just was responding to your issue since you asked. I think in practice its not a big difference since I’m very poorly/barely maintaining it now, anyways…

@sagikazarmark Extremely supportive of forking this under Viper. Viper is a great project. I’d be happy to update the README and support that as the official, I think that project has shown good OSS stewardship.

And I also agree with the linked comments that a fork is the best option, rather than a transfer.

sagikazarmark commented 11 months ago

Thank you @prochac for opening this issue.

@mitchellh I forked the project under go-viper (https://github.com/go-viper/mapstructure). (There isn't much in there yet, but that's where Viper subprojects will gravitate eventually)

Thank you for all the work you've put into this project (well...all of them). I wish you all the best in your next endeavor!

mitchellh commented 11 months ago

Thank you very much. I'll leave the project unarchived for now and let you handle migrating any issues or PRs you want. Then at some future point I'll update the README to point to the Viper project and archive.

❤️

sagikazarmark commented 11 months ago

Thank you @mitchellh !

I started pulling in some of the more serious bug fixes into the fork.

I'll tag 1.6.0 soon with the current module name and then tag a v2 renaming the module, so the fixes can still be used as a drop-in replacement.

I also plan to send a message to all pending issues and PRs asking them to resubmit in the new repo, so that we can do some cleanup around stale issues/PRs.

sagikazarmark commented 11 months ago

Tagged a new version with the old import path: https://github.com/go-viper/mapstructure/releases/tag/v1.6.0

Also started to work on v2 with the new import path: https://github.com/go-viper/mapstructure/releases/tag/v2.0.0-alpha.1

mitchellh commented 11 months ago

@sagikazarmark I mistakenly thought the go-viper org and Viper project in reference this whole time was the official spf13/viper project. I can see you're the top contributor to spf13/viper, and I assume maintainer but not 100% sure. Can you make it clear what your association is and what the go-viper org is? (I'm still thankful for your fork and work here and I'm not accusing you of anything untoward, just want to make sure this is totally clarified)

sagikazarmark commented 11 months ago

@mitchellh no worries!

Yes, I'm a maintainer of Viper (spf13/viper) and the go-viper organization is associated with that project. The long-term plan is to move some of the remote providers and encoding plugins to external repos to reduce the number of dependencies in the core Viper project.

The core repository will remain under @spf13's personal account to avoid breaking the thousands of imports. :)

Hope that clarifies things a bit.

prochac commented 11 months ago

@mitchellh that's why I added the link to viper contributors. https://github.com/spf13/viper/graphs/contributors To add @sagikazarmark some credibility. He seems legit.

mitchellh commented 11 months ago

All good, thanks to both you 😄

prochac commented 11 months ago

But is open source, anybody can be sleeping russian agent

mitchellh commented 11 months ago

I hid your comment, I don't want to offend anyone right now as we're just about to archive this project 😄

sagikazarmark commented 11 months ago

@mitchellh are you OK with me sending a message to all open issues and PRs that they may resubmit to the fork?

I pulled in the PRs the seemed to have fixed the most inportant issues, but there are quite a few other valuable contributions there.

mitchellh commented 11 months ago

Yes I’m okay with that, thanks for asking

thaJeztah commented 11 months ago

Curious if transferring the repository is an option, so that GitHub's redirects lead visitors to the right place, preserve more history such as tickets and pull requests, and go modules to function without "replace" rules for versions of the module before renaming.

I see the other repository is now "ahead" a couple of commits, but the module hasn't been renamed yet, so that may complicate things a bit (this repository's git would probably have to be aligned before that)

thaJeztah commented 11 months ago

I see the other repository is now "ahead" a couple of commits, but the module hasn't been renamed yet

Ignore me 🙈 I looked at the v1.6.0 tag, which carries the old name, but it was renamed since

sagikazarmark commented 11 months ago

@thaJeztah no worries, your question prompted me to add a migration guide to the README.

EronWright commented 9 months ago

Any guidance on which repo to open issues into?

sagikazarmark commented 9 months ago

@EronWright https://github.com/go-viper/mapstructure is the blessed fork, so I'd suggest opening new issues there.

Thiht commented 7 months ago

If the change is official, can you add a notice to the README, and maybe archive this repository? The current state of affairs is a bit confusing tbh

sagikazarmark commented 6 months ago

I tagged 2.0.0: https://github.com/go-viper/mapstructure/releases/tag/v2.0.0

sagikazarmark commented 6 months ago

@mitchellh I can open a PR to add a notice to the readme if you want me to, but it may be better if it comes from you. Let me know which one you prefer.