Skip to content
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.

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

Fix a panic when deriving Deserialize for an enum with tuple and struct with flatten field #2291

Closed

Conversation

Mingun
Copy link
Contributor

@Mingun Mingun commented Oct 5, 2022

Fixes #1904

@Mingun

This comment was marked as abuse.

@oli-obk

This comment was marked as resolved.

@Mingun

This comment was marked as off-topic.

@dtolnay
Copy link
Member

dtolnay commented Oct 14, 2022

Is there a particular person you feel has been doing a good job providing code reviews of serde PRs to the point of perhaps having earned merge permission? It definitely doesn't seem like it to me but I am still interested in your opinion @Mingun.

As far as my own time, it always gets prioritized according to what is most impactful, which currently happens to be my Wasm proc macros RFC a.k.a. make serde_derive compile in 0.0 seconds!

I can say "please do not periodically commit anything to serde unless all pull requests are being reviewed" is not a reasonable perspective. The latter is not a goal at the expense of dedicating time in more impactful ways.

@dtolnay
Copy link
Member

dtolnay commented Oct 14, 2022

In the end, the community has repeatedly offered assistance in maintaining the project, but maintainers do not just lack time to do this -- they do not want someone else to do this

I went and reread both of the threads you linked fully, and it turns out there is not one single person in either of them offering to help in maintenance, so your framing on this is extremely disingenuous. The nearest thing is you @Mingun "volunteering" other people to do the work, which is..... not how this works. On top of this, framing the only developer willing to participate in the work as "dog in the manger" is not a productive contribution to the situation. I've set a 30-day ban for you over this but I would love for you to contribute productively after that point if you are still willing.

@dtolnay dtolnay closed this Oct 14, 2022
@Mingun
Copy link
Contributor Author

Mingun commented Feb 25, 2023

It is very sad that instead of listening to criticism, you are scaling your ego. Your behavior looks especially low when you shut your opponent's mouth to confirm your point of view.

there is not one single person in either of them offering to help in maintenance, so your framing on this is extremely disingenuous.

I believed that the initial steps of a third-party developer who wants to help the project were to do PRs and inquire about the status of the project. If the owners / maintainers recognize the difficulties and express a desire to get help, then you can already offer it. I have not seen such a desire.

The lack of information from official maintainers (unfortunately, besides you, it is not clear who this is, since GitHub does not provide this information and it is not written anywhere) that they need help, even if there are two issues, which indicated that something is not good in the current situation, I regard it as that it is not required by them. Whereas all issues / PRs are usually ignored, I do not understand the ways in which you even assume that someone would offer help, even if they could.

Besides, I do not see any disingenuous from my side if I correctly understand meaning of the "assistance" word (which, I think, you use as indicator to blame me in disingenuous). Making PRs fixing bugs is fall into the category of assistance, don't you think? Of course, it would be better, if I'd write "try to assist". Sorry for that, it is not so easely to me as non-native speaker to discuss on not technical themes, where I could have lack of some typical patterns. I do not consider this an excuse for the use of administrative levers.

On top of this, framing the only developer willing to participate in the work as "dog in the manger" is not a productive contribution to the situation.

Right now I see 42 open PRs from 38 people what I can not call "the only developer willing to participate in the work". Ok, some of them opened several years ago, but at least 7 was opened this year (from 7 different people), so I would say, that here has the active developers.

I would love for you to contribute productively after that point if you are still willing.

I would be glad, but unfortunately it does not depend on me, as eloquently evidenced by the closed, but not merged status of this PR, which itself does not depend on our off-topic.

@zeenix
Copy link

zeenix commented Jun 3, 2023

I went and reread both of the threads you linked fully, and it turns out there is not one single person in either of them offering to help in maintenance

As I commented 2 years prior to your comment here, this is a chicken&egg problem. You won't just give merge rights to the project to the first person who volunteers. So it has to be a contributor and if important PRs don't get merged or get even a review for years, how can someone step up for the job?

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

Successfully merging this pull request may close these issues.

Deserialize failed to derive on enums with tuple variant and flatten struct variant
4 participants