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 a test for verify_tarball #4097

Merged
merged 3 commits into from
Oct 28, 2021
Merged

Add a test for verify_tarball #4097

merged 3 commits into from
Oct 28, 2021

Conversation

nipunn1313
Copy link
Contributor

Depends on #4096.

(Because of community/community#4477 - it doesn't render the diff against #4096 - so you have to go to commits tab to see appropriate diff).

@Turbo87 Turbo87 added A-backend ⚙️ C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear labels Oct 28, 2021
@Turbo87
Copy link
Member

Turbo87 commented Oct 28, 2021

@bors r+

@bors
Copy link
Contributor

bors commented Oct 28, 2021

📌 Commit b25ad21 has been approved by Turbo87

@bors
Copy link
Contributor

bors commented Oct 28, 2021

⌛ Testing commit b25ad21 with merge f76943f...

@bors
Copy link
Contributor

bors commented Oct 28, 2021

☀️ Test successful - checks-actions
Approved by: Turbo87
Pushing f76943f to master...

@bors bors merged commit f76943f into rust-lang:master Oct 28, 2021
@nipunn1313 nipunn1313 deleted the tarball_up2 branch October 30, 2021 19:10
bors added a commit that referenced this pull request Jan 3, 2022
Support relative URLs from repo-subdirectory packages

Extract `.cargo_vcs_info.json` from the tarball and use it to determine the path in vcs.

Support for this was added to cargo with
rust-lang/cargo#9866

Fixes #3484
(note that support for this in admin::render_readmes not yet exists - would come with #4095)

Depends on #4097 (Because of community/community#4477 - github ends up rendering the diffs together. Go to the commits tab and just look at the most recent commits to review).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-backend ⚙️ C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants