Merge #4101

4101: Panic proc macro srv if read request failed r=matklad a=edwin0cheng

This PR fixed a bug when the rust-analyzer is killed suddenly, the `rust-analyzer proc-macro` will become stale.

Co-authored-by: Edwin Cheng <edwin0cheng@gmail.com>

Refactor a bit

Merge #4118

4118: :arrow_up: crates (to bust the caches) r=matklad a=matklad

bors r+
🤖

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

:arrow_up: crates (to bust the caches)

Merge #4117

4117: Honor snippet capability r=matklad a=matklad

closes #2518

bors r+
🤖

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

Honor snippet capability

closes #2518

Merge #4116

4116: Make sure that adding a snippet requires corresponding capability r=matklad a=matklad

bors r+
🤖

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

Make sure that adding a snippet requires corresponding capability

Introduce internal snippet cap

Move CompletionConfig to a separate module

Merge #4114

4114: Refactor completion sorting r=matklad a=matklad

bors r+
🤖

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

Don't set sortText

I might be reading this wrong, but it looks like we are setting it to
essentially arbitrary string at the moment, as there are no defined
order on the items in the *set* of completions.

Add test marks

Introduce ActiveParameter

Move tests to where they belong

Refactor

Restore CompletionItem immutability

More functional

Move

Merge #4106

4106: Fix wrong substitution code r=matklad a=flodiebold

We need to shift in when we're substituting inside a binder.

This should fix #4053 (it doesn't fix the occasional overflow that also occurs on the Diesel codebase though).

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

Fix wrong substitution code

We need to shift in when we're substituting inside a binder.

This should fix #4053 (it doesn't fix the occasional overflow that also occurs
on the Diesel codebase though).

Merge #3954

3954: Improve autocompletion by looking on the type and name r=matklad a=bnjjj

This tweet (https://twitter.com/tjholowaychuk/status/1248918374731714560) gaves me the idea to implement that in rust-analyzer.

Basically for this first example I made some examples when we are in a function call definition. I look on the parameter list to prioritize autocompletions for the same types and if it's the same type + the same name then it's displayed first in the completion list.

So here is a draft, first step to open a discussion and know what you think about the implementation. It works (cf tests) but maybe I can make a better implementation at some places. Be careful the code needs some refactoring to be better and concise.

PS: It was lot of fun writing this haha

Co-authored-by: Benjamin Coenen <5719034+bnjjj@users.noreply.github.com>

Merge #4111 #4112

4111: Record definitions in `extern` blocks r=jonas-schievink a=jonas-schievink

Enables completion of extern functions and statics.

Closes https://github.com/rust-analyzer/rust-analyzer/issues/3711

4112: Add Launch configuration for release build r=matklad a=jonas-schievink

The debug build takes very long until I can test anything useful, with the release build it's much quicker. Add another Run configuration for it.

Co-authored-by: Jonas Schievink <jonasschievink@gmail.com>

Record definitions in `extern` blocks

Add Launch configuration for release build

Merge #4110

4110: minor r=matklad a=matklad

bors r+
🤖

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

minor

Merge #4108

4108: Fully get rid of SyntaxNodePtr::range r=matklad a=matklad

bors r+
🤖

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

Fully get rid of SyntaxNodePtr::range

Merge #4107

4107: Disable onEnter command by default r=matklad a=matklad

We are transitioning from experimental to production-ready stance, so
it makes sense to disable potentially disruptive features by default.

bors r+
🤖

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

Disable onEnter command by default

We are transitioning from experimental to production-ready stance, so
it makes sense to disable potentially disruptive features by default.

Bubble up error

Merge #4102

4102: Cleanup proc_macro config r=matklad a=matklad

In general, there should be no reason to call `.to_string_lossy`.
If you want to display the path, use `.display()`.
If you want to pass the path to an OS API (like std::process::Command)
than use `PathBuf` or `OsString`.

bors r+
🤖

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

Cleanup proc_macro config

In general, there should be no reason to call `.to_string_lossy`.
If you want to display the path, use `.display()`.
If you want to pass the path to an OS API (like std::process::Command)
than use `PathBuf` or `OsString`.

Merge #4100

4100: Bump required rust to 1.43 r=matklad a=kjeremy

This only impacts source builds

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

Fix typo

Panic proc macro srv if read request failed

Bump required rust to 1.43

Merge #4065

4065: Complete unqualified enum names in patterns and expressions r=matklad a=nathanwhit

This PR implements the completion described in #4014.
The result looks like so for patterns:
<img width="542" alt="Screen Shot 2020-04-20 at 3 53 55 PM" src="https://user-images.githubusercontent.com/17734409/79794010-8f529400-831f-11ea-9673-f838aa9bc962.png">

and for `expr`s:
<img width="620" alt="Screen Shot 2020-04-21 at 3 51 24 PM" src="https://user-images.githubusercontent.com/17734409/79908784-d73ded80-83e9-11ea-991d-921f0cb27e6f.png">

I'm not confident that the completion text itself is very robust, as it will unconditionally add completions for enum variants with the form `Enum::Variant`. This means (I believe) it would still suggest `Enum::Variant` even if the local name is changed i.e. `use Enum as Foo` or the variants are brought into scope such as through `use Enum::*`.

Co-authored-by: nathanwhit <nathan.whitaker01@gmail.com>

Merge #4099

4099: Remove SyntaxPtr::range from more places r=matklad a=matklad

bors r+
🤖

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

Remove SyntaxPtr::range from more places

Merge #4093

4093: Simplify config r=matklad a=Veetaha

Co-authored-by: veetaha <veetaha2@gmail.com>

Merge #4094

4094: proc_macro: add ability to log to stderr and view output in vscode r=matklad a=Veetaha

r? @edwin0cheng

Co-authored-by: veetaha <veetaha2@gmail.com>

Merge #4092

4092: feat: run ignored tests r=matklad a=hdevalke

I started making some exercices on https://exercism.io/ and a lot of test have the `#[ignore]` attribute.
The `Run Test|Debug` code lens show up, but running the test results in:

```
running 1 test
test test_one_piece ... ignored

test result: ok. 0 passed; 0 failed; 1 ignored; 0 measured; 5 filtered out
```

This pull request adds the `--ignored` flag if needed.

Co-authored-by: Hannes De Valkeneer <hannes@de-valkeneer.be>

Merge #4098

4098: Add YouCompleteMe as a LSP option for vim/neovim r=matklad a=weihanglo

As YouCompleteMe supports [arbitrary LSP implementation](https://ycm-core.github.io/YouCompleteMe/#plugging-an-arbitrary-lsp-server), it could be added to the list of vim LSP implementation. It works like a charm.

Co-authored-by: Weihang Lo <weihanglo@users.noreply.github.com>

Add YouCompleteMe as a LSP option for vim/neovim

Merge #4096

4096: tasks.json Support r=matklad a=jcdickinson

Move the task provider anonymous class into a real class, as this seems to be how Microsoft do this in their documentation.

resolveTask is now implemented, which is used by VSCode to determine how to execute tasks that the user has defined in tasks.json.

Resolves #3983

Co-authored-by: Jonathan Dickinson <jonathanD@k2.com>

Merge #4095

4095: Fix typo r=edwin0cheng a=Veetaha

Co-authored-by: veetaha <veetaha2@gmail.com>

tasks.json Support

Move the task provider anonymous class into a real class, as this seems
to be how Microsoft do this in their documentation.

resolveTask is now implemented, which is used by VSCode to determine how
to execute tasks that the user has defined in tasks.json.

Fix typo

proc_macro: add ability to log to stderr and view output in vscode

Simplify config

feat: run ignored tests

Update tests to reflect new completions

Add tests for enum completion

Adds tests for completion of enum variants in match arms, if-let statements, and basic expressions.

Complete unqualified enum variants when possible

Add utility fn for expected type of a node

Adds `expected_type_of` to `CompletionContext` to return the expected type of a
node, if it is known.

Merge #4090

4090: Fix config naming r=matklad a=matklad

bors r+
🤖

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

Align the name of proc-macro enabling flag

We use `enable`, not `enabled` elsewhere

Match implementation of diagnostics flag to the docs

Merge #4089

4089: npm update r=kjeremy a=kjeremy

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

npm update

Merge #4088

4088: cargo update r=kjeremy a=kjeremy

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

cargo update

Merge #4082

4082: Don't panic on `rust-analyzer --help` r=matklad a=eminence

Special case handling for `-h` and `--help` if no subcommand is given.

Closes #4068

Co-authored-by: Andrew Chin <achin@eminence32.net>

Merge #4081

4081: Work around crlf in syntax tree r=matklad a=Veetaha

Workarounds fixes #4067

Co-authored-by: veetaha <veetaha2@gmail.com>

Refactor the workaround a bit

Work around crlf in syntax tree

Merge #4080

4080: Fix .rast tmGrammar to account for numbers in identifiers r=matklad a=Veetaha

Co-authored-by: veetaha <veetaha2@gmail.com>

Don't panic on `rust-analyzer --help`

Special case handling for `-h` and `--help` if no subcommand is given.

Closes #4068

Fix .rast tmGrammar to account for numbers in identifiers

Merge #4078

4078: Do not add default and closure types in 'add explicit type' assist r=matklad a=SomeoneToIgnore

Co-authored-by: Kirill Bulatov <mail4score@gmail.com>

Do not add default and closure types in 'add explicit type' assist

Merge #4076

4076: Improve remove derive attrs r=matklad a=edwin0cheng

This PR implemented a proper `remove_derive_attrs` function which **merely** remove  any `#[derive(*)]` attributes.

Co-authored-by: Edwin Cheng <edwin0cheng@gmail.com>

Improve remove dervie attrs

Improve autocompletion by looking on the type and name

Signed-off-by: Benjamin Coenen <5719034+bnjjj@users.noreply.github.com>

Merge #4038

4038: Group generated ast boilerplate apart from the interesting part r=matklad a=Veetaha

Boilerplate `AstNode` and `From` impls are moved to the end further from the interesting part in `generated.rs`

Co-authored-by: veetaha <veetaha2@gmail.com>

Improve autocompletion by looking on the type and name

Signed-off-by: Benjamin Coenen <5719034+bnjjj@users.noreply.github.com>

Merge branch 'master' of github.com:rust-analyzer/rust-analyzer

Improve autocompletion by looking on the type and name

Signed-off-by: Benjamin Coenen <5719034+bnjjj@users.noreply.github.com>

Merge #4071

4071: Cleanup args a bit r=matklad a=matklad

bors r+
🤖

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

Cleanup args a bit

Merge #4070

4070: docs(readme): improve user docs r=matklad a=fannheyward

cleanup unavailable configurations/commands

Co-authored-by: Heyward Fann <fannheyward@gmail.com>

docs(readme): improve user docs

cleanup unavailable configurations/commands

Merge #4069

4069: More detailed Sublime Text install instructions r=lnicola a=nnmm

* People might typically jump directly to their editor and wonder where the part about installing rust-analyzer is – at least I did. I added a link to the relevant section for ST.
* Make ST instructions more detailed and user friendly (especially beginners), include troubleshooting tips.
* Minor grammar improvements throughout.

Co-authored-by: Nikolai Morin <nnmmgit@gmail.com>

Move the PATH issue up to the non-editor specific section

Delete comma

Update docs/user/readme.adoc

Co-Authored-By: Laurențiu Nicola <lnicola@users.noreply.github.com>

Update docs/user/readme.adoc

Co-Authored-By: Laurențiu Nicola <lnicola@users.noreply.github.com>

More detailed Sublime Text install instructions

* People might typically jump directly to their editor and wonder where the part about installing rust-analyzer is – at least I did. I added a link to the relevant section for ST.
* Make ST instructions more detailed and user friendly (especially beginners), include troubleshooting tips.
* Minor grammar improvements throughout.

Merge #4066

4066: Fix restart missing arguments in proc-macro-srv r=edwin0cheng a=edwin0cheng

cc @Veetaha

Co-authored-by: Edwin Cheng <edwin0cheng@gmail.com>

Refactor a bit

Fix restart missing arguments in proc-macro-srv

Merge #4061

4061: ra_proc_macro: cleanups here and there r=edwin0cheng a=Veetaha

r? @edwin0cheng

Co-authored-by: veetaha <veetaha2@gmail.com>
Co-authored-by: Veetaha <veetaha2@gmail.com>

Migrate to Result<T, io::Error> -> io::Result<T>

Fix doc comment

Fix typo

Co-Authored-By: Laurențiu Nicola <lnicola@users.noreply.github.com>

Display path not debug it

Co-Authored-By: Laurențiu Nicola <lnicola@users.noreply.github.com>

Display path not debug it

Co-Authored-By: Laurențiu Nicola <lnicola@users.noreply.github.com>

ra_proc_macro: cleanups here and there