-
Notifications
You must be signed in to change notification settings - Fork 13.1k
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
Emit crate disambiguators in save-analysis data #45468
Conversation
Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @nrc (or someone else) soon. If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes. Please see the contribution instructions for more information. |
☔ The latest upstream changes (presumably #45096) made this pull request unmergeable. Please resolve the merge conflicts. |
r+ once the rls-data PR lands and using the new rls-data version. |
Resolved conflict and updated version using rls-data 0.12 |
@Xanewok Is this PR still [WIP], or is it ready to be merged? 🙂 |
@bors: r+ |
📌 Commit 45fa7b2 has been approved by |
⌛ Testing commit 45fa7b2 with merge e0108b32a832227c30ffdb285fe2162e85dba3fa... |
💔 Test failed - status-appveyor |
@nrc this failed on compiling rls (rls-data mismatch). What should we do here to resolve this? |
Updating rls submodule to rust-45468 branch, so it can compile with these changes. |
@bors: r+ |
📌 Commit 930464e has been approved by |
⌛ Testing commit 930464e1884b351fbd1664597a5869cb94dfc432 with merge 3e2d634482aba5a01ee874915c1798107d7d90aa... |
💔 Test failed - status-appveyor |
single test in rls failed:
|
@bors: r+ |
📌 Commit b4ffede has been approved by |
⌛ Testing commit b4ffede with merge b56c5189726be531377b3aa39b3ea98b4f214f43... |
💔 Test failed - status-travis |
|
|
Emit crate disambiguators in save-analysis data Needed for rust-dev-tools/rls-analysis#93. Blocked by rust-dev-tools/rls-data#11. (For now, this pulls my branch [rls-data/crate-source](https://github.com/Xanewok/rls-data/tree/crate-source)) This will allow to disambiguate different crates types/versions when indexing resulting save-analysis data (most importantly allow to support bin+lib and different crate versions). r? @nrc
☀️ Test successful - status-appveyor, status-travis |
Needed for rust-dev-tools/rls-analysis#93.
Blocked by rust-dev-tools/rls-data#11. (For now, this pulls my branch rls-data/crate-source)
This will allow to disambiguate different crates types/versions when indexing resulting save-analysis data (most importantly allow to support bin+lib and different crate versions).
r? @nrc