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

Remove Badges from the UI #2440

Merged
merged 4 commits into from
Apr 25, 2020
Merged

Remove Badges from the UI #2440

merged 4 commits into from
Apr 25, 2020

Conversation

Turbo87
Copy link
Member

@Turbo87 Turbo87 commented Apr 18, 2020

Resolves #2436

@rust-highfive
Copy link

r? @carols10cents

(rust_highfive has picked a reviewer for you, use r? to override)

@locks
Copy link
Contributor

locks commented Apr 25, 2020

@bors r+

This was brought up at the weekly meeting and there seemed to be general agreement with the direction!

@bors
Copy link
Contributor

bors commented Apr 25, 2020

📌 Commit a9f6708 has been approved by locks

@bors
Copy link
Contributor

bors commented Apr 25, 2020

⌛ Testing commit a9f6708 with merge 8421062...

@bors
Copy link
Contributor

bors commented Apr 25, 2020

☀️ Test successful - checks-travis
Approved by: locks
Pushing 8421062 to master...

@bors bors merged commit 8421062 into rust-lang:master Apr 25, 2020
@Turbo87 Turbo87 deleted the remove-badges branch April 25, 2020 14:43
@Turbo87 Turbo87 removed the A-ui label Apr 25, 2020
bors added a commit to rust-lang/cargo that referenced this pull request Sep 23, 2020
Remove some badges documentation.

Badges have been removed from crates.io, so this updates the documentation. I don't think Cargo should manage a schema for the different services, and since crates.io isn't using these, I think it is mostly a dead feature. I left the `maintenance` field, because that might still have some meaning in the future.

More details:
* Removal from crates.io: rust-lang/crates.io#2436 rust-lang/crates.io#2440
* Solicited feedback for this change: https://internals.rust-lang.org/t/cargo-badges/12982
* Potential future support of `maintenance` status: rust-lang/crates.io#2437 rust-lang/crates.io#2438 rust-lang/crates.io#2439. It's not clear, if crates.io manages the status in the database, the motivation for putting it in `Cargo.toml` is probably pretty small.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove badges from crate lists and details
5 participants