Skip to content

report private repository error as broken crate #453

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

Merged
merged 3 commits into from
Sep 24, 2019

Conversation

zeegomo
Copy link
Contributor

@zeegomo zeegomo commented Sep 7, 2019

Change crate result from Error to Broken if the crate repo is private or not available.

Copy link
Member

@pietroalbini pietroalbini left a comment

Choose a reason for hiding this comment

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

Can we also add a minicrater test to ensure private repos are probably detected? https://github.com/ghost/doesnt-exist is a good candidate for it.

@pietroalbini pietroalbini self-assigned this Sep 9, 2019
@zeegomo zeegomo force-pushed the broken-private-repo branch from dad8ceb to 2dc801d Compare September 16, 2019 16:10
Copy link
Member

@pietroalbini pietroalbini left a comment

Choose a reason for hiding this comment

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

Thanks! I'll hold off merging until rustwide 0.3.0 is released so we can remove the git dependency, but this looks great!

@bors
Copy link
Contributor

bors commented Sep 23, 2019

☔ The latest upstream changes (presumably #462) made this pull request unmergeable. Please resolve the merge conflicts.

@zeegomo zeegomo force-pushed the broken-private-repo branch from 2dc801d to 443bdf8 Compare September 23, 2019 16:28
@pietroalbini
Copy link
Member

Thanks!

@bors r+

@bors
Copy link
Contributor

bors commented Sep 24, 2019

📌 Commit 443bdf8 has been approved by pietroalbini

@bors
Copy link
Contributor

bors commented Sep 24, 2019

⌛ Testing commit 443bdf8 with merge 4f2ecdd...

bors added a commit that referenced this pull request Sep 24, 2019
report private repository error as broken crate

Change crate result from `Error` to `Broken` if the crate repo is private or not available.
@bors
Copy link
Contributor

bors commented Sep 24, 2019

💔 Test failed - checks-travis

@pietroalbini
Copy link
Member

@bors retry

@bors
Copy link
Contributor

bors commented Sep 24, 2019

⌛ Testing commit 443bdf8 with merge 59d58f5...

bors added a commit that referenced this pull request Sep 24, 2019
report private repository error as broken crate

Change crate result from `Error` to `Broken` if the crate repo is private or not available.
@bors
Copy link
Contributor

bors commented Sep 24, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: pietroalbini
Pushing 59d58f5 to master...

@bors bors merged commit 443bdf8 into rust-lang:master Sep 24, 2019
@zeegomo zeegomo deleted the broken-private-repo branch May 14, 2020 18:05
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.

3 participants