Add a comment

Fix snap

Add a bunch of new documentation to completions

Add way of getting docs from the code model and use for completion

Make EnumVariant a DocCommentsOwner

Merge #617 #618

617: Use npm ci instead of install r=matklad a=h-michael

fix #422

`npm install` is always recreate `package-lock.json`.
So we might use `npm ci` with `install-code`

https://docs.npmjs.com/cli/ci.html#description

618: Add install-lsp subcommand r=matklad a=h-michael

related https://github.com/rust-analyzer/rust-analyzer/pull/617#issuecomment-456914650

Co-authored-by: Hirokazu Hata <h.hata.ai.t@gmail.com>

Merge #619

619: More correct raw ident handling r=matklad a=jrobsonchase

Also added the suggested test from #616.

It passed before the change to `scan_ident`, but at least now it should ensure that the behavior stays the same.

Co-authored-by: Josh Robson Chase <josh@robsonchase.com>

More correct raw ident handling

Add install-lsp subcommand

Merge #616

616: Lex raw idents correctly r=matklad a=jrobsonchase

One question:

What's the intent of the text by the `IDENT` node under `NAME`? Should it be the actual token text or the semantic name?

Closes #611

Co-authored-by: Josh Robson Chase <josh@robsonchase.com>

Use npm ci instead of install

Use IDENT for both raw and normal idents

Add raw idents to lexer and parser

Merge #615

615: Bumpup lsp-types to v0.55.0 r=DJMcNab a=h-michael

https://github.com/gluon-lang/lsp-types/compare/v0.54.0...v0.55.0

Co-authored-by: Hirokazu Hata <h.hata.ai.t@gmail.com>

Fix target_selection_range type

Bumpup lsp-types to v0.55.0

drop obsolete comment

Merge #614

614: Add supported LSP features to README.md r=matklad a=h-michael

Co-authored-by: Hirokazu Hata <h.hata.ai.t@gmail.com>

Add supported LSP features to README.md

Merge #613

613: assign DefIds when lowering r=matklad a=matklad

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

assign DefIds when lowering

Merge #612

612: Some random refactorings r=matklad a=matklad

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

introduced better typed AstPtr

refactor

simiplify

better names

Merge #610

610: move SyntaxPtr to ra_syntax r=matklad a=matklad

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

move SyntaxPtr to ra_syntax

Merge #608

608: Complete parens r=matklad a=matklad

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

don't insert parens for self methods

move completion item tests closer to the code

this is the reason why we need marks: the tests were spread across two
files, because I've forgotten that there were tests already

Merge #606

606: generalize marking infrastructure r=matklad a=matklad

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

generalize marking infrastructure

Merge #605

605: :arrow_up: insta r=matklad a=matklad

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

:arrow_up: insta

Merge #603

603: fix completion bugs r=gfreezy a=gfreezy

The result is as below.

before:

after:

Co-authored-by: gfreezy <gfreezy@gmail.com>

fix completion bugs

Merge #598

598: Add function signature to CompletionItem detail r=matklad a=kjeremy

Co-authored-by: Jeremy A. Kolb <jkolb@ara.com>

Move label from hir to ide_api

Merge #602

602: add status command r=matklad a=matklad

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

document status command

ad status command

Merge #601

601: Simplify CallInfo label and documentation r=matklad a=kjeremy

Co-authored-by: Jeremy A. Kolb <jkolb@ara.com>

Simplify CallInfo label and documentation

Merge #600

600: :arrow_up: rowan r=matklad a=matklad

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

:arrow_up: rowan

Merge #599

599: Allow placeholder parameters in trait function definitions r=matklad a=regiontog

fixes #597

Co-authored-by: Erlend Tobiassen <erlend.tobiassen@gmail.com>

Add test for placeholder parameters in trait fn defs

Allow placeholder parameters in trait fn defs

Add function signature to CompletionItem detail

Merge #593

593: Docs for completion r=matklad a=kjeremy

The first commit adds documentation support to CompletionItems.

The second one I am unsure about. Is that the right way to add docs for functions? If so should I do something similar for other `hir` types and CompletionItems?

Co-authored-by: Jeremy Kolb <kjeremy@gmail.com>
Co-authored-by: Jeremy A. Kolb <jkolb@ara.com>

Update snaps

Move docs to Function

Merge #592

592: Allow types to the left of `:` in where predicates r=matklad a=regiontog

Adresses #584

Co-authored-by: Erlend Tobiassen <erlend.tobiassen@gmail.com>

No need for is_type_start

Prefer TYPE_FIRST

Update tests after allowing where predicate to accept types

Merge #594

594: fold macros r=matklad a=matklad

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

fold macros

Optimistically bail out of where clause loop if not at start of a type or lifetime

Thread documentation through FnSignature and CompletionItem

Teach CompletionItem about documentation

Don't leave a marker hanging without completing it.

Allow types to the left of : in where predicates.

Merge #591

591: :arrow_up: insta r=matklad a=matklad

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

:arrow_up: insta

Merge #590

590: Use insta for ty tests r=matklad a=flodiebold

Co-authored-by: Florian Diebold <flodiebold@gmail.com>

Merge #588

588: Fix .not postfix completion r=matklad a=flodiebold

The postfix snippets are awesome! I don't think the `.not` one is supposed to always replace the receiver by literally `!not` :smile:

Co-authored-by: Florian Diebold <flodiebold@gmail.com>

Use insta for ty tests

Fix .not postfix completion

Merge #586

586: Fix panic trying to get substs on unknown type r=matklad a=flodiebold

Fixes #585.

Co-authored-by: Florian Diebold <flodiebold@gmail.com>

Fix panic trying to get substs on unknown type

Fixes #585.

mention guide in architecture

add illustrations

Apply suggestions from code review

Co-Authored-By: matklad <aleksey.kladov@gmail.com>

create a reference cycle by linking the video

reach fixed-point for rust-analyzer spelling

fix a ton of typos

fix the tag

complete completion section

start completions walkthrough

write about type inference

source map pattern

guide about recursive macto magic

start chapter about interners

finish modules section

add guide

Merge #582

582: Postfix completion r=matklad a=gfreezy

resolved #525

Co-authored-by: gfreezy <gfreezy@gmail.com>

Merge #581

581: Typo fix r=DJMcNab a=gfreezy

Co-authored-by: gfreezy <gfreezy@gmail.com>

postfix completion

typos fix

Merge #574

574: refactor completions to use TextEdit instead of InsertText r=matklad a=gfreezy

1. migrate from `insertText` to `TextEdit` from `CompleteItem`
2. use `insta` to test completions

Co-authored-by: gfreezy <gfreezy@gmail.com>

Merge #580

580: extend selection expands macros and can totally panic r=matklad a=matklad

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

extend selection expands macros and can totally panic

somewhat better name

make matching brace consistent

use with_db consistently

Merge #579

579: Fix panic on inferring field access on an enum r=matklad a=flodiebold

Seen while skipping through https://youtu.be/ANKBNiSWyfc ;)

Co-authored-by: Florian Diebold <flodiebold@gmail.com>

Fix panic on inferring field access on an enum

fix tests