Merge #11152

11152: internal: add more tests for entry points r=matklad a=matklad

bors r+
🤖

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

add path tests

add expr tests

tests for ty

internal: add tests for patterns

Merge #11144

11144: fix: Fix server path on NixOS r=lnicola a=lnicola

See https://github.com/rust-analyzer/rust-analyzer/commit/c0d0ceefb29111732cacf39dbb57fe9f3ce25ee8#commitcomment-62688259.

bors r+

Co-authored-by: Laurențiu Nicola <lnicola@dend.ro>

Fix server path on NixOS

Merge #11141

11141: internal: add tests for extra parser entry points r=matklad a=matklad

bors r+
🤖

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

Merge #10692

10692: Migrate to new rowan r=lnicola a=azdavis

Since https://github.com/rust-analyzer/rowan/pull/122 was merged into rowan, this could be how rust-analyzer migrates.

Co-authored-by: Ariel Davis <ariel.z.davis@icloud.com>

Migrate to SyntaxNodePtr in new rowan

restore invariatns

parse empty statemet as statemetn

Add test

rename

compress

add automated semicolon insertion test

internal: add tests for extra parser entry points

Merge #11134

11134: internal: tighten up parser API r=matklad a=matklad

It's tempting to expose things like `Expr::parse`,
but they'll necessary have somewhat ad-hoc semantics.

Instead, we narrow down the parser's interface strictly
to what's needed for MBE. For everything else (eg, parsing
imports), the proper way is enclose the input string into
some context, parse the whole as a file, and then verify
that the input was parsed as intended.

bors r+
🤖

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

compress

dead code

introduce hacks module

clarify semantics of doc links

last use of parse api in ssr

dead code

remove fragments from syntax

move the rest of ssr parsing to fragments

drop dead code

add ssr fragment for statements

add ssr fragment for expressions

verify during parse

add test

remove Item::parse

internal: start isolating ssr-related parsing APIs to SSR

add missing test

dead code

add TopEntryPoint

simplify

move path

move expr

move ty

move pat to prefix entry points

move stmt to entry points

internal: move block to prefix entry point

internal: move visibility to a prefix entry point

internal: add prefix entry points

cleanup imports

simplify

Merge #11131

11131: internal: avoid speculation when completing macros r=matklad a=matklad

bors r+
🤖

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

minor: simplify

minor: dedup

avoid speculation when completing macros

Merge #11128

11128: import style r=matklad a=matklad

bors r+
🤖

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

import style

Merge #11127

11127: internal: Set a timeout on the Octokit client r=lnicola a=lnicola

CC #11056

Co-authored-by: Laurențiu Nicola <lnicola@dend.ro>

Set a timeout on the Octokit client

Merge #11125

11125: minor r=matklad a=matklad

bors r+
🤖

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

minor

Merge #11122

11122: Remove note about alpha status from vscode extension descritpion r=lnicola a=aDotInTheVoid

See https://github.com/rust-analyzer/rust-analyzer.github.io/pull/156 and https://github.com/rust-analyzer/rust-analyzer/pull/8237

Co-authored-by: Nixon Enraght-Moony <nixon.emoony@gmail.com>

Remove note about alpha status from vscode extension descritpion

See https://github.com/rust-analyzer/rust-analyzer.github.io/pull/156 and https://github.com/rust-analyzer/rust-analyzer/pull/8237

Merge #11120

11120: internal: move parser tests to parser crate r=matklad a=matklad

bors r+
🤖

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

fix line endings

internal: move inline parser tests to parser crate

internal: move outlined parser tests

Merge #11118

11118: internal: move ws attachment logic to the parser crate r=matklad a=matklad

This has to re-introduce the `sink` pattern, because doing this purely
with iterators is awkward :( Maaaybe the event vector was a false start?

But, anyway, I like the current factoring more -- it sort-of obvious
that we do want to keep ws-attachment business in the parser, and that
we also don't want that to depend on the particular tree structure. I
think `shortcuts` module achieves that.

bors r+
🤖

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

internal: move ws attachment logic to the parser crate

This has to re-introduce the `sink` pattern, because doing this purely
with iterators is awkward :( Maaaybe the event vector was a false start?

But, anyway, I like the current factoring more -- it sort-of obvious
that we do want to keep ws-attachment business in the parser, and that
we also don't want that to depend on the particular tree structure. I
think `shortcuts` module achieves that.

Merge #11117

11117: internal: replace TreeSink with a data structure  r=matklad a=matklad

The general theme of this is to make parser a better independent
library.

The specific thing we do here is replacing callback based TreeSink with
a data structure. That is, rather than calling user-provided tree
construction methods, the parser now spits out a very bare-bones tree,
effectively a log of a DFS traversal.

This makes the parser usable without any *specifc* tree sink, and allows
us to, eg, move tests into this crate.

Now, it's also true that this is a distinction without a difference, as
the old and the new interface are equivalent in expressiveness. Still,
this new thing seems somewhat simpler. But yeah, I admit I don't have a
suuper strong motivation here, just a hunch that this is better.

cc #10765

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

rename

internal: rename

internal: replace TreeSink with a data structure

The general theme of this is to make parser a better independent
library.

The specific thing we do here is replacing callback based TreeSink with
a data structure. That is, rather than calling user-provided tree
construction methods, the parser now spits out a very bare-bones tree,
effectively a log of a DFS traversal.

This makes the parser usable without any *specifc* tree sink, and allows
us to, eg, move tests into this crate.

Now, it's also true that this is a distinction without a difference, as
the old and the new interface are equivalent in expressiveness. Still,
this new thing seems somewhat simpler. But yeah, I admit I don't have a
suuper strong motivation here, just a hunch that this is better.

Merge #11116

11116: :arrow_up: expect test r=matklad a=matklad

bors r+
🤖

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

:arrow_up: expect test

Merge #10933

10933: show values of constants in hover r=lnicola a=HKalbasi

Fix #8497
cc #8655

Co-authored-by: hkalbasi <hamidrezakalbasi@protonmail.com>

show values of constants in hover

Merge #11106

11106: internal: Remove network access from Code extension r=lnicola a=lnicola

Co-authored-by: Laurențiu Nicola <lnicola@dend.ro>

Spelling fix

Simplify conflict check

Drop kalitaalexey.vscode-rust check

Don't check for platform details

Update privacy note

Remove ETXTBUSY check, we don't overwrite binaries

Improve NixOS handling

Drop extensionUri copy

Remove proxy settings

Remove channel and ask before download prefs

Remove node-fetch and https-proxy-agent deps

Remove network functions

Remove server download and update checks

Merge #11103

11103: internal: Improve `github-release` action r=lnicola a=lnicola

Upgrade ``@actions/github`` to get `listReleaseAssets` and retry individual uploads in addition to the whole thing.

Also disables `ci`, `metrics` and `rustdoc` on forks to save some CPU time.

CC #11056

Co-authored-by: Laurențiu Nicola <lnicola@dend.ro>

Gate workflows to only run on upstream

Improve github-release action

Merge #11102

11102: minor: Update known issues in the bug report template r=lnicola a=lnicola

bors r+

Co-authored-by: Laurențiu Nicola <lnicola@dend.ro>

Update known issues in the bug report template

Merge #11096

11096: Code: Use the same `displayName` for pre-release versions r=lnicola a=lnicola

Otherwise the Marketplace shows `(nightly)` for the stable version.

bors r+

Co-authored-by: Laurențiu Nicola <lnicola@dend.ro>

Use the same displayName for pre-release versions

Merge #11095

11095: internal: Disable unstable APIs for nightly releases r=lnicola a=lnicola

bors r+

The Marketplace doesn't allow us to publish extensions using these, even as pre-release versions.

Co-authored-by: Laurențiu Nicola <lnicola@dend.ro>

Disable unstable APIs for nightly releases

Merge #11094

11094: Actually nightly releases to Matketplace r=lnicola a=lnicola

bors r+

Co-authored-by: Laurențiu Nicola <lnicola@dend.ro>

Actually nightly releases to Matketplace

Merge #11093

11093: fix: Do not complete `Drop::drop`, complete `std::mem::drop` instead r=Veykril a=Veykril

Fixes https://github.com/rust-analyzer/rust-analyzer/issues/5005
bors r+

Co-authored-by: Lukas Wirth <lukastw97@gmail.com>

Make hover test not relient on minicore structure