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

Amend RFC 1242 to specify what “minimal maintenance” means for r-l-deprecated crates #2624

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jethrogb
Copy link
Contributor

@jethrogb jethrogb force-pushed the deprecation-maintenance branch from aed6722 to 17fc0d9 Compare January 13, 2019 12:12
@Centril Centril added T-libs-api Relevant to the library API team, which will review and decide on the RFC. A-nursery Proposals relating to the rust-lang-nursery. A-governance Proposals relating to how Rust is governed. labels Jan 13, 2019
@KodrAus
Copy link
Contributor

KodrAus commented Jan 13, 2019

Having a better understanding of what it means to deprecate a nursery crate seems like a good goal to me!

The main concern I have right now is I don't think this RFC sufficiently acknowledges our real lack of bandwidth for some nursery crates, let alone those in the deprecated organisation. I think bandwidth is the main reason some of these libraries receive no maintenance. Reviewing, merging, and publishing even small PRs in a repository that you aren't immersed in has a lot of peripheral context building effort in addition to that needed to review, merge, and publish. Having a policy for maintenance doesn't create bandwidth for what is, unfortunately, a bit of a thankless task.

On the other hand though, if there are folks interested in supporting these libraries then having a framework for helping them decide what level of contribution to consider might help them onboard.

@sfackler
Copy link
Member

I am also curious who exactly you think is going to be performing this work.

@jethrogb
Copy link
Contributor Author

In a sense, I think the question of who should be performing the maintenance is orthogonal to this RFC. The status quo already calls for maintenance. This RFC just specifies what's included in that. If there are currently deprecated crates without maintainers, that situation should be resolved by the crate owner/libs team/community ASAP.

Perhaps you are implicitly advocating for no maintenance. As mentioned in the drawbacks/alternatives section of this RFC, I don't think that's a defensible position.

@jethrogb
Copy link
Contributor Author

jethrogb commented Jan 14, 2019

Reviewing, merging, and publishing even small PRs in a repository that you aren't immersed in has a lot of peripheral context building effort in addition to that needed to review, merge, and publish.

I don't agree with the “peripheral context building effort” part. Especially for small PRs, often no context is needed. Looking at the most recent merges of time and rustc-serialize, most of these could've been easily reviewed by anyone familiar with Rust but not at all familiar with the crate itself.

@KodrAus
Copy link
Contributor

KodrAus commented Jan 14, 2019

In a sense, I think the question of who should be performing the maintenance is orthogonal to this RFC.

I get the sentiment here, and while it doesn't seem unreasonable I think reality undermines the goals of this RFC. So I have to disagree that the question of who should perform the maintenance is orthogonal. It's fundamental. Without addressing that root problem we're really just elaborating on a process that doesn't actually exist outside the text in this repository.

That being said, I'm not opposed to this, I just think we need to understand the picture it fits into better, manage our expectations around how maintenance on deprecated repositories is likely to be performed, and look at how changes/clarifications to the deprecation process could result in an increase in bandwidth for these kinds of tasks.

@Ericson2314
Copy link
Contributor

We should really look at the dependency graph. Maybe there's just a few popularish crates still using these that bump up the download counts.

@varkor
Copy link
Member

varkor commented Jan 15, 2019

I think setting out explicit guidelines as to how much maintenance a crate will receive is valuable. Declaring a crate to be "unmaintained", even if that's an undesirable stance, is better than claiming it's "minimally maintained" at the moment, when even minimal pull requests are not accepted.

If it's actually not enough bandwidth to maintain these crates, it's better to be up front about this. Part of the frustration with the status quo is setting incorrect expectations.

(To be clear, I'm not advocating one alternative or the other here; just that being clear about what deprecation does entail is one of the most important factors.)

@gnzlbg
Copy link
Contributor

gnzlbg commented Oct 15, 2019

If there are currently deprecated crates without maintainers, that situation should be resolved by the crate owner/libs team/community ASAP.

@jethrogb How ? Finding maintainers even for non-deprecated crates is hard. The crate owner in this case would be the rust-lang org, and beyond asking whether someone wants to maintain a particular crate, there is not much that can be done AFAICT.

@jethrogb
Copy link
Contributor Author

@gnzlbg I'm not convinced that this is going to be an issue. If someone wants something to change about a crate without a maintainer, they are incentivized to find a maintainer. It could be that same person as well.

@KodrAus KodrAus added the Libs-Tracked Libs issues that are tracked on the team's project board. label Jul 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-governance Proposals relating to how Rust is governed. A-nursery Proposals relating to the rust-lang-nursery. Libs-Tracked Libs issues that are tracked on the team's project board. T-libs-api Relevant to the library API team, which will review and decide on the RFC.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants