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.

Sign up for GitHub

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

Jump to bottom

Disable fast submodule checkout due to spurious ci failures #106682

Closed
wants to merge 1 commit into from
Closed

Disable fast submodule checkout due to spurious ci failures #106682

wants to merge 1 commit into from

Conversation

pietroalbini
Copy link
Member

Same fix we included in #98423.

@rustbot
Copy link
Collaborator

rustbot commented Jan 10, 2023

r? @jyn514

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

@rustbot rustbot added A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. labels Jan 10, 2023
@pietroalbini pietroalbini mentioned this pull request Jan 10, 2023
Closed
@JohnTitor
Copy link
Member

@bors r+ p=51

@bors
Copy link
Contributor

bors commented Jan 10, 2023

📌 Commit 4b782a8 has been approved by JohnTitor

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Jan 10, 2023

🌲 The tree is currently closed for pull requests below priority 50. This pull request will be tested once the tree is reopened.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jan 10, 2023
@bors
Copy link
Contributor

bors commented Jan 10, 2023

⌛ Testing commit 4b782a8 with merge f66ea871a5ad531bcf43cb226db8c240977d5ede...

This was referenced Jan 10, 2023
Merged
Merged
@pietroalbini
Copy link
Member Author

@bors r- retry

Included in the CVE fixes PRs.

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Jan 10, 2023
@pietroalbini pietroalbini deleted the pa-disable-fast-submodule-checkout branch January 10, 2023 16:49
@rust-log-analyzer
Copy link
Collaborator

A job failed! Check out the build log: (web) (plain)

Click to see the possible cause of the failure (guessed by this bot)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JohnTitor JohnTitor JohnTitor approved these changes

Assignees

@jyn514 jyn514

Labels
A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@pietroalbini @rustbot @JohnTitor @bors @rust-log-analyzer @jyn514