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.

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

rust-analyzer: use in-tree pattern_analysis crate #128922

Merged
merged 1 commit into from
Aug 16, 2024

Conversation

Nadrieril
Copy link
Member

@rustbot
Copy link
Collaborator

rustbot commented Aug 10, 2024

Failed to set assignee to Veykril: invalid assignee

Note: Only org members with at least the repository "read" role, users with write permissions, or people who have commented on the PR may be assigned.

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 10, 2024
@rustbot
Copy link
Collaborator

rustbot commented Aug 10, 2024

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

@Nadrieril Nadrieril force-pushed the r-a-use-upstream-pat-ana branch from eecad10 to d340d51 Compare August 10, 2024 12:55
@Veykril
Copy link
Member

Veykril commented Aug 10, 2024

Failed to set assignee to Veykril: invalid assignee

Note: Only org members with at least the repository "read" role, users with write permissions, or people who have commented on the PR may be assigned.

Shouldn't I qualify for that first one? 🤔

@rust-log-analyzer

This comment has been minimized.

@Veykril
Copy link
Member

Veykril commented Aug 10, 2024

r? @Veykril

@Nadrieril Nadrieril force-pushed the r-a-use-upstream-pat-ana branch from d340d51 to acb0ecc Compare August 10, 2024 13:40
@Veykril
Copy link
Member

Veykril commented Aug 10, 2024

@bors r+

@bors
Copy link
Contributor

bors commented Aug 10, 2024

📌 Commit acb0ecc has been approved by Veykril

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 Aug 10, 2024
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Aug 11, 2024
…, r=Veykril

rust-analyzer: use in-tree `pattern_analysis` crate

r? `@Veykril`
@matthiaskrgr
Copy link
Member

@bors r-
needs rebase/ does not build

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Aug 11, 2024
@Veykril
Copy link
Member

Veykril commented Aug 13, 2024

rust-lang/rust-analyzer#17853 fixes the mismatches on our side now, so we should probably switch this to in-tree once that is merged and we've synced the subtree again

@Nadrieril Nadrieril marked this pull request as draft August 14, 2024 16:24
@Nadrieril
Copy link
Member Author

Makes sense. Marking as draft in the meantime.

@lnicola
Copy link
Member

lnicola commented Aug 14, 2024

The sync PR got merged today, we should be good to go?

@Nadrieril
Copy link
Member Author

Oh, perfect!

@Nadrieril Nadrieril marked this pull request as ready for review August 14, 2024 18:52
@Nadrieril Nadrieril force-pushed the r-a-use-upstream-pat-ana branch from acb0ecc to bb84372 Compare August 14, 2024 18:52
@Veykril
Copy link
Member

Veykril commented Aug 15, 2024

@bors r+

@bors
Copy link
Contributor

bors commented Aug 15, 2024

📌 Commit bb84372 has been approved by Veykril

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-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Aug 15, 2024
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Aug 15, 2024
…, r=Veykril

rust-analyzer: use in-tree `pattern_analysis` crate

r? `@Veykril`
bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 15, 2024
…llaumeGomez

Rollup of 8 pull requests

Successful merges:

 - rust-lang#128348 (Unconditionally allow shadow call-stack sanitizer for AArch64)
 - rust-lang#128922 (rust-analyzer: use in-tree `pattern_analysis` crate)
 - rust-lang#128935 (More work on `zstd` compression)
 - rust-lang#129072 (Infer async closure args from `Fn` bound even if there is no corresponding `Future` bound on return)
 - rust-lang#129101 (Fix projections when parent capture is by-ref but child capture is by-value in the `ByMoveBody` pass)
 - rust-lang#129106 (Remove redundant type ops: `Eq`/`Subtype`)
 - rust-lang#129122 (Remove duplicated `Rustdoc::output` method from `run-make-support` lib)
 - rust-lang#129124 (rustdoc-json: Use FxHashMap from rustdoc_json_types)

r? `@ghost`
`@rustbot` modify labels: rollup
workingjubilee added a commit to workingjubilee/rustc that referenced this pull request Aug 16, 2024
…, r=Veykril

rust-analyzer: use in-tree `pattern_analysis` crate

r? `@Veykril`
workingjubilee added a commit to workingjubilee/rustc that referenced this pull request Aug 16, 2024
…, r=Veykril

rust-analyzer: use in-tree `pattern_analysis` crate

r? ``@Veykril``
bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 16, 2024
…kingjubilee

Rollup of 10 pull requests

Successful merges:

 - rust-lang#128064 (Improve docs for Waker::noop and LocalWaker::noop)
 - rust-lang#128922 (rust-analyzer: use in-tree `pattern_analysis` crate)
 - rust-lang#128965 (Remove `print::Pat` from the printing of `WitnessPat`)
 - rust-lang#128977 (Only try to modify file times of a writable file on Windows)
 - rust-lang#129018 (Migrate `rlib-format-packed-bundled-libs` and `native-link-modifier-bundle` `run-make` tests to rmake)
 - rust-lang#129037 (Port `run-make/libtest-json` and `run-make/libtest-junit` to rmake)
 - rust-lang#129078 (`ParamEnvAnd::fully_perform`: we have an `ocx`, use it)
 - rust-lang#129110 (Add a comment explaining the return type of `Ty::kind`.)
 - rust-lang#129111 (Port the `sysroot-crates-are-unstable` Python script to rmake)
 - rust-lang#129135 (crashes: more tests)

r? `@ghost`
`@rustbot` modify labels: rollup
bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 16, 2024
…kingjubilee

Rollup of 9 pull requests

Successful merges:

 - rust-lang#128064 (Improve docs for Waker::noop and LocalWaker::noop)
 - rust-lang#128922 (rust-analyzer: use in-tree `pattern_analysis` crate)
 - rust-lang#128965 (Remove `print::Pat` from the printing of `WitnessPat`)
 - rust-lang#129018 (Migrate `rlib-format-packed-bundled-libs` and `native-link-modifier-bundle` `run-make` tests to rmake)
 - rust-lang#129037 (Port `run-make/libtest-json` and `run-make/libtest-junit` to rmake)
 - rust-lang#129078 (`ParamEnvAnd::fully_perform`: we have an `ocx`, use it)
 - rust-lang#129110 (Add a comment explaining the return type of `Ty::kind`.)
 - rust-lang#129111 (Port the `sysroot-crates-are-unstable` Python script to rmake)
 - rust-lang#129135 (crashes: more tests)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 4fd4da9 into rust-lang:master Aug 16, 2024
6 checks passed
@rustbot rustbot added this to the 1.82.0 milestone Aug 16, 2024
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request Aug 16, 2024
Rollup merge of rust-lang#128922 - Nadrieril:r-a-use-upstream-pat-ana, r=Veykril

rust-analyzer: use in-tree `pattern_analysis` crate

r? ```@Veykril```
@Nadrieril Nadrieril deleted the r-a-use-upstream-pat-ana branch September 2, 2024 19:05
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.

8 participants