Skip to content
This repository has been archived by the owner on Dec 29, 2022. It is now read-only.

Update racer, rustfmt and clippy #1313

Merged
merged 1 commit into from
Feb 14, 2019
Merged

Update racer, rustfmt and clippy #1313

merged 1 commit into from
Feb 14, 2019

Conversation

kngwyu
Copy link
Contributor

@kngwyu kngwyu commented Feb 14, 2019

No description provided.

@kngwyu kngwyu changed the title Update racer, rustfmt, clippy Update racer, rustfmt and clippy Feb 14, 2019
@Xanewok
Copy link
Member

Xanewok commented Feb 14, 2019

@bors r+

@bors
Copy link
Contributor

bors commented Feb 14, 2019

📌 Commit c78c953 has been approved by Xanewok

@bors
Copy link
Contributor

bors commented Feb 14, 2019

⌛ Testing commit c78c953 with merge 7cd58a4...

bors added a commit that referenced this pull request Feb 14, 2019
Update racer, rustfmt and clippy
@bors
Copy link
Contributor

bors commented Feb 14, 2019

💔 Test failed - checks-travis

@Xanewok
Copy link
Member

Xanewok commented Feb 14, 2019 via email

@bors
Copy link
Contributor

bors commented Feb 14, 2019

💡 This pull request was already approved, no need to approve it again.

  • This pull request previously failed. You should add more commits to fix the bug, or use retry to trigger a build again.

@bors
Copy link
Contributor

bors commented Feb 14, 2019

📌 Commit c78c953 has been approved by Xanewok

@bors
Copy link
Contributor

bors commented Feb 14, 2019

⌛ Testing commit c78c953 with merge 9d917fe...

bors added a commit that referenced this pull request Feb 14, 2019
Update racer, rustfmt and clippy
@Xanewok
Copy link
Member

Xanewok commented Feb 14, 2019 via email

@bors
Copy link
Contributor

bors commented Feb 14, 2019

💡 This pull request was already approved, no need to approve it again.

  • This pull request is currently being tested. If there's no response from the continuous integration service, you may use retry to trigger a build again.

@bors
Copy link
Contributor

bors commented Feb 14, 2019

📌 Commit c78c953 has been approved by Xanewok

@bors
Copy link
Contributor

bors commented Feb 14, 2019

⌛ Testing commit c78c953 with merge 0d6f53e...

bors added a commit that referenced this pull request Feb 14, 2019
Update racer, rustfmt and clippy
@bors
Copy link
Contributor

bors commented Feb 14, 2019

☀️ Test successful - checks-travis
Approved by: Xanewok
Pushing 0d6f53e to master...

@bors bors merged commit c78c953 into rust-lang:master Feb 14, 2019
@jens1o
Copy link
Contributor

jens1o commented Feb 14, 2019

Will this make it possible that rls will be in the next nightly cycle again?

@Xanewok
Copy link
Member

Xanewok commented Feb 14, 2019 via email

@mati865
Copy link
Contributor

mati865 commented Feb 14, 2019

Many people don't fully understand how it works.
Rust doesn't care about Clippy and Rustfmt versions in RLS repository because it patches RLS to use version that is checked at /src/tools/{clippy,rustfmt}.

So changes to them are not necessary to unblock tools for nightly. However it fixes stand-alone builds of RLS.

Racer update on the other hand is necessary because it syncs rustc-ap-syntax version to match other packages.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants