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

Add maintenance badge docs #4431

Merged
merged 1 commit into from
Aug 24, 2017
Merged

Add maintenance badge docs #4431

merged 1 commit into from
Aug 24, 2017

Conversation

vignesh-sankaran
Copy link
Contributor

@vignesh-sankaran vignesh-sankaran commented Aug 24, 2017

Pending merging of this PR for crates.io. This should close off #704 on crates.io. I've also updated the badge metadata docs and reordered the fields into groups by build, code coverage, and maintenance.

So I've decided to put the list of possible badge options in the documentation, we'll have to find another place to put more detailed descriptions of the available maintenance badges.

@alexcrichton
Copy link
Member

@bors: r+

Thanks!

@bors
Copy link
Contributor

bors commented Aug 24, 2017

📌 Commit 5cafa5a has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Aug 24, 2017

⌛ Testing commit 5cafa5a with merge 69fa59a...

bors added a commit that referenced this pull request Aug 24, 2017
…crichton

Add maintenance badge docs

Pending merging of [this PR for crates.io](rust-lang/crates.io#952). This should close off [#704 on crates.io](rust-lang/crates.io#704). I've also updated the badge metadata docs and reordered the fields into groups by build, code coverage, and maintenance.

So I've decided to put the list of possible badge options in the documentation, we'll have to find another place to put more detailed descriptions of the available maintenance badges.
@bors
Copy link
Contributor

bors commented Aug 24, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 69fa59a to master...

@bors bors merged commit 5cafa5a into rust-lang:master Aug 24, 2017
@vignesh-sankaran vignesh-sankaran deleted the maintenance-badge-docs branch August 25, 2017 06:48
bors added a commit that referenced this pull request Apr 19, 2019
Improved docs for `maintenance` options

I thought the lack of clear meanings for the available options could use some improvement.
Basically an upgrade of #4431.

Got the relevant text from rust-lang/crates.io#704 and the corresponding [rfc](https://github.com/rust-lang/rfcs/blob/master/text/1824-crates.io-default-ranking.md)

*Questions*
1. Should I put the descriptions outside of the code block?
2. I copied the wording including "maintainer" and "author", should I replace it with "you" or something more appropriate?
@ehuss ehuss added this to the 1.21.0 milestone Feb 6, 2022
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.

4 participants