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

[CratesMSRV] Badge #9871

Merged
merged 1 commit into from Jan 7, 2024
Merged

[CratesMSRV] Badge #9871

merged 1 commit into from Jan 7, 2024

Conversation

Sky9x
Copy link
Contributor

@Sky9x Sky9x commented Jan 3, 2024

Proposed in #9733

Routes:
Crates.io MSRV: /crates/msrv/:crate
Crates.io MSRV (version): /crates/msrv/:crate/:version

Copy link
Contributor

github-actions bot commented Jan 3, 2024

Messages
📖 ✨ Thanks for your contribution to Shields, @Sky9x!

Generated by 🚫 dangerJS against 9c1986d

@chris48s chris48s added the service-badge Accepted and actionable changes, features, and bugs label Jan 4, 2024
@Sky9x
Copy link
Contributor Author

Sky9x commented Jan 5, 2024

failure is not in a file that i modified?

EDIT:

failing test case:

// in crates-downloads.tester.js
t.create('downloads (invalid version)')
  .get('/d/libc/7.json')
  .expectBadge({ label: 'crates.io', message: 'invalid semver: 7' })

the actual message was "not found"

no clue how this happened

Copy link
Contributor

github-actions bot commented Jan 6, 2024

🚀 Updated review app: https://pr-9871-badges-shields.fly.dev

@chris48s chris48s changed the title [Crates] MSRV Badge [CratesMSRV] Badge Jan 6, 2024
Copy link
Member

@chris48s chris48s left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for picking this up. Looks like the test failure is unrelated to the changes in this PR. I've set this PR to just run the tests for MSRV. We can deal with addressing that test in another PR.

services/crates/crates-msrv.service.js Show resolved Hide resolved
services/crates/crates-msrv.service.js Outdated Show resolved Hide resolved
services/crates/crates-msrv.service.js Outdated Show resolved Hide resolved
services/crates/crates-msrv.service.js Outdated Show resolved Hide resolved
@Sky9x
Copy link
Contributor Author

Sky9x commented Jan 7, 2024

should be good to go unless you have any more suggestions

Copy link
Contributor

github-actions bot commented Jan 7, 2024

🚀 Updated review app: https://pr-9871-badges-shields.fly.dev

Crates.io MSRV: `/crates/msrv/:crate`
Crates.io MSRV (version): `/crates/msrv/:crate/:version`
@chris48s chris48s added this pull request to the merge queue Jan 7, 2024
Merged via the queue into badges:master with commit 57aaaad Jan 7, 2024
22 checks passed
@Sky9x Sky9x deleted the crates-msrv branch January 7, 2024 21:00
@chris48s chris48s mentioned this pull request Jan 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
service-badge Accepted and actionable changes, features, and bugs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants