Skip to content

Subtree update of rust-analyzer #142250

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

Merged
merged 62 commits into from
Jun 10, 2025
Merged

Subtree update of rust-analyzer #142250

merged 62 commits into from
Jun 10, 2025

Conversation

lnicola
Copy link
Member

@lnicola lnicola commented Jun 9, 2025

r? @ghost

aibaars and others added 30 commits May 30, 2025 22:38
Co-authored-by: Lukas Wirth <me@lukaswirth.dev>
Produce ClosureBinder node in atom.rs
The Rust source code previously mised ProjectJson and JsonProject,
and the TypeScript definition didn't match either.

Use 'ProjectJson' everywhere. This is purely an internal change.
internal: Use 'ProjectJson' consistently
…riginal symbol

Co-authored-by: Jake Goulding <jake.goulding@integer32.com>
Enhance renaming to include identifiers that are generated from the original symbol
Enable Assist edit for tuple<->named struct for the struct and visiblity keywords
fix: Cycle handlers for `HirDatabase::infer, const_param_ty_with_diagnostics`
Add a quickfix for accessing a private field of a struct
Bumps [tar-fs](https://github.com/mafintosh/tar-fs) from 2.1.2 to 2.1.3.
- [Commits](https://github.com/mafintosh/tar-fs/commits)

---
updated-dependencies:
- dependency-name: tar-fs
  dependency-version: 2.1.3
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Revert `cargo-workspaces` version fix
ci: When autopublishing, remove `xtask` from workspace
Veykril and others added 16 commits June 5, 2025 15:04
fix: Record macro calls in signatures in `ChildBySource` impls
Better parser recovery for macro calls in type bound position
…-stable

fix: Stabilize unlinked file diagnostic
…-quickfix

fix: Always include quickfixes for diagnostics, even when diagnostics are disabled
fix: Record macro calls for fields in `ChildBySource` impls
…attribute_completions

feat: implement attribute completions for diagnostics module
fix: Stabilize the "JSON is not Rust" diagnostic
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jun 9, 2025
@rustbot
Copy link
Collaborator

rustbot commented Jun 9, 2025

rust-analyzer is developed in its own repository. If possible, consider making this change to rust-lang/rust-analyzer instead.

cc @rust-lang/rust-analyzer

@lnicola
Copy link
Member Author

lnicola commented Jun 9, 2025

@bors r+ p=1 subtree sync

@bors
Copy link
Collaborator

bors commented Jun 9, 2025

📌 Commit f5bfde2 has been approved by lnicola

It is now in the queue for this repository.

@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 9, 2025
@bors
Copy link
Collaborator

bors commented Jun 10, 2025

⌛ Testing commit f5bfde2 with merge 8a407a8...

@bors
Copy link
Collaborator

bors commented Jun 10, 2025

☀️ Test successful - checks-actions
Approved by: lnicola
Pushing 8a407a8 to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Jun 10, 2025
@bors bors merged commit 8a407a8 into rust-lang:master Jun 10, 2025
11 checks passed
@rustbot rustbot added this to the 1.89.0 milestone Jun 10, 2025
Copy link
Contributor

What is this? This is an experimental post-merge analysis report that shows differences in test outcomes between the merged PR and its parent PR.

Comparing c6768de (parent) -> 8a407a8 (this PR)

Test differences

No test diffs found

Test dashboard

Run

cargo run --manifest-path src/ci/citool/Cargo.toml -- \
    test-dashboard 8a407a82848bbc926de1cbbbbcb381e1a96f5968 --output-dir test-dashboard

And then open test-dashboard/index.html in your browser to see an overview of all executed tests.

Job duration changes

  1. dist-aarch64-linux: 5769.0s -> 7776.5s (34.8%)
  2. x86_64-apple-2: 3710.7s -> 4636.1s (24.9%)
  3. aarch64-apple: 4598.4s -> 3721.4s (-19.1%)
  4. dist-apple-various: 9413.9s -> 7796.1s (-17.2%)
  5. dist-x86_64-apple: 11889.8s -> 10255.2s (-13.7%)
  6. dist-ohos-x86_64: 4573.3s -> 4045.0s (-11.6%)
  7. dist-aarch64-apple: 5423.1s -> 4854.2s (-10.5%)
  8. dist-ohos-armv7: 4338.5s -> 3899.9s (-10.1%)
  9. dist-s390x-linux: 5197.4s -> 4710.4s (-9.4%)
  10. dist-powerpc64le-linux-gnu: 5377.7s -> 4886.7s (-9.1%)
How to interpret the job duration changes?

Job durations can vary a lot, based on the actual runner instance
that executed the job, system noise, invalidated caches, etc. The table above is provided
mostly for t-infra members, for simpler debugging of potential CI slow-downs.

@lnicola lnicola deleted the sync-from-ra branch June 10, 2025 08:15
@rust-timer
Copy link
Collaborator

Finished benchmarking commit (8a407a8): comparison URL.

Overall result: no relevant changes - no action needed

@rustbot label: -perf-regression

Instruction count

This benchmark run did not return any relevant results for this metric.

Max RSS (memory usage)

This benchmark run did not return any relevant results for this metric.

Cycles

Results (secondary -1.8%)

A less reliable metric. May be of interest, but not used to determine the overall result above.

mean range count
Regressions ❌
(primary)
- - 0
Regressions ❌
(secondary)
- - 0
Improvements ✅
(primary)
- - 0
Improvements ✅
(secondary)
-1.8% [-1.8%, -1.8%] 1
All ❌✅ (primary) - - 0

Binary size

This benchmark run did not return any relevant results for this metric.

Bootstrap: 755.17s -> 753.608s (-0.21%)
Artifact size: 372.30 MiB -> 372.36 MiB (0.02%)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged-by-bors This PR was explicitly merged by bors. 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.