Skip to content

Update RLS to disable spurious client_find_definitions test#62259

Closed
Xanewok wants to merge 1 commit intorust-lang:masterfrom
Xanewok:update-rls
Closed

Update RLS to disable spurious client_find_definitions test#62259
Xanewok wants to merge 1 commit intorust-lang:masterfrom
Xanewok:update-rls

Conversation

@Xanewok
Copy link
Copy Markdown
Contributor

@Xanewok Xanewok commented Jun 30, 2019

Since switching CI to Azure Pipelines it seems that this test seems
to fail more consistently, so let's disable that for now. It helps
that we have less than a week before release - we disallow PRs that
break the tools to land in this period, so this makes landing critical
PRs smoother now.

r? @alexcrichton

Since switching CI to Azure Pipelines it seems that this test seems
to fail more consistently, so let's disable that for now. It helps
that we have less than a week before release - we disallow PRs that
break the tools to land in this period, so this makes landing critical
PRs smoother  now.

r? @alexcrichton
@rust-highfive
Copy link
Copy Markdown
Contributor

⚠️ Warning ⚠️

  • These commits modify submodules.

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jun 30, 2019
@Mark-Simulacrum
Copy link
Copy Markdown
Member

@bors r+

@bors
Copy link
Copy Markdown
Collaborator

bors commented Jun 30, 2019

📌 Commit 639e03b has been approved by Mark-Simulacrum

@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 Jun 30, 2019
@Mark-Simulacrum
Copy link
Copy Markdown
Member

@bors p=5

@Mark-Simulacrum
Copy link
Copy Markdown
Member

@bors p=0

cherry-picked into #62247

Centril added a commit to Centril/rust that referenced this pull request Jul 1, 2019
Update RLS to disable spurious client_find_definitions test

Since switching CI to Azure Pipelines it seems that this test seems
to fail more consistently, so let's disable that for now. It helps
that we have less than a week before release - we disallow PRs that
break the tools to land in this period, so this makes landing critical
PRs smoother  now.

r? @alexcrichton
@emilyalbini
Copy link
Copy Markdown
Member

Already merged into #62247.

@emilyalbini emilyalbini closed this Jul 1, 2019
@Xanewok Xanewok deleted the update-rls branch July 1, 2019 19:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants