Merge #10902

10902: Handle multiple cargo check quick fix spans r=Veykril a=brandondong

Resolves https://github.com/rust-analyzer/rust-analyzer/issues/10705.

**Cause:**
- For a cargo check diagnostic with multiple spans, only a single quick fix action would be created at the location of `spans[0]`. Additionally, the hover window details would only show the location of `spans[0]` next to the message.

**Fix:**
- Allow cargo check quick fix actions to be triggerable from multiple selection ranges. Specifically, if the selection intersects with any of the replacement spans, the quick fix action is shown.
- No change in behavior for the hover window details. It's pretty minor and I think showing multiple locations next to the message may be more confusing anyways.

Co-authored-by: Brandon <brandondong604@hotmail.com>

Update expected test results

Handle multiple cargo check quick fix spans

Refactor away unnecessary Vec

Merge #10930

10930: fix: Fix self keyword not being tagged as such in highlighting properly r=Veykril a=Veykril

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

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

fix: Fix self keyword not being tagged as such in highlighting properly

Merge #10929

10929: internal: Split up macro/attribute semantic tokens a bit more r=Veykril a=Veykril

Fixes https://github.com/rust-analyzer/rust-analyzer/issues/9172

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

Update semantic token docs

Introduce SymbolKind::Derive

Introduce SymbolKind::Attribute

Merge #10928

10928: minor: Bump deps r=lnicola a=lnicola

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

Bump pulldown-cmark-to-cmark

Bump miow

Bump countme

Bump chalk

Bump most deps

Merge #10927

10927: minor: bump the rest of the npm deps r=lnicola a=lnicola

bors r+

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

Bump rest of the npm deps

Bump node-fetch

Bump vsce

Bump eslint

Bump engine in lockfile

Merge #10925

10925: minor: Simplify `.vscodeignore` r=lnicola a=lnicola

The quadruple negation we used to have here was quite confusing:

```gitignore
**
!out
out/**
!out/src
```

Let's ignore everything and cherry-pick what we want to include.

~~And also include the extension sourcemap, just in case someone wants to debug the extension.~~

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

Simplify `.vscodeignore`

Merge #10924

10924: minor: Add LICENSE file to make vsce happy r=lnicola a=lnicola

Newer `vsce` versions error out if a `LICENSE` file is missing, even if the license is specified in `package.json`. Concatenate the two licenses we use and add a file with that to make a future `vsce` upgrade easier.

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

Merge #10926

10926: minor: Bump Node to 14 in release workflow r=lnicola a=lnicola

bors r+

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

Bump Node to 14 in release workflow

Add LICENSE file to make vsce happy

Merge #10921

10921: minor: Update list of safe intrinsics r=Veykril a=Veykril

Closes https://github.com/rust-analyzer/rust-analyzer/issues/4798
bors r+

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

minor: Update list of safe intrinsics

Merge #10920

10920: minor: Shorten spans of mismatched_arg_count diag r=Veykril a=Veykril

Fixes #5289
bors r+

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

minor: Shorten spans of mismatched_arg_count diag

Merge #10919

10919: internal: Define missing semantic token hierarchies in package.json r=Veykril a=Veykril

Closes https://github.com/rust-analyzer/rust-analyzer/issues/10077
bors r+

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

Add some more semantic token to textmate grammar mapping

Define missing semantic token hierarchies in package.json

Merge #10916

10916: feat: ide: Convert while let to loop r=Veykril a=rainy-me

close #10901

Co-authored-by: rainy-me <github@yue.coffee>

fix: defer condition construction

Merge #10915

10915: feat: Resolve builtin-attr and tools in ide layer r=Veykril a=Veykril

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

Improve hover message for inert attributes

Merge #10917

10917: minor: Bump some npm deps r=lnicola a=lnicola

bors r+

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

Bump engine

Bump esbuild

Bump typescript

Bump @types/vscode

Bump eslint

Bump vscode-test

Bump tslib

Bump node-fetch and @types/node-fetch

Bump @types/node

Bump d3

feat: handle while let to loop

Rename things: Tool -> ToolModule

Basic hover for builtin-attr and tool modules

Simplify

Remove syntax highlighting hack for builtin attrs

feat: Resolve builtin-attr and tools in ide layer

Merge #10906

10906: [first contributation] fix: `add return type` assit works when there's missing whitespace r=Veykril a=izik1

I feel like the way I handled whitespace here isn't... Right? Maybe it should be folded into `InsertOrReplace::Insert`

Also, sorry about the commit name, I am _not_ good at writing user facing commit messages

Also sorry about the test names, could be clearer I feel

Co-authored-by: Skyler Rain Ross <orangesnowfox@gmail.com>

refactor(assist/add_return_type): avoid threading `needs_whitespace`

Merge #10903

10903: internal: Replace `mocha` with a custom test runner r=Aloso a=lnicola

Closes #10723

r? `@Aloso`

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

fix: add return type works when there's missing whitespace

Merge #10905

10905: minor: Fix proxyStrictSSL check r=lnicola a=lnicola

bors r+

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

Fix proxyStrictSSL check

Merge #10891

10891: fix: emit trait names in moniker identifier r=Veykril a=tjdevries

Hi,

Not sure if this is the best fix for the problem, but I noticed that functions that part of a trait are not namespaced with the trait name, so this could lead to problems if you had a function and a trait-function both defined in the same module.

Thanks!

TJ

Co-authored-by: TJ DeVries <devries.timothyj@gmail.com>

Replace mocha with a custom test runner

fixup: include more information for impls

fixup: properly handle all associated items

Merge #10899

10899: ide: hack to make self not unresolved reference in async trait wrapped impl's r=Veykril a=jhgg

fixes #10708

this is a bit hacky, but it "works".

i'm not sure how to even write a test for this though, but i've confirmed it works via manual testing...

Co-authored-by: Jake Heinz <jh@discordapp.com>

add a fixme:

simplify

ide: hack to make self not unresolved reference in async trait wrapped impl's

Merge #10897

10897: internal: Try to fix Code tests r=lnicola a=lnicola

It looks like `--noEmit` broke our unit tests:


r? `@ChayimFriedman2`

I suppose we might still:

 - compile stuff twice
 - not detect failing tests on CI (since we didn't notice this)

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

Bump to Node 14

Use xvfb-run directly

Bump xvfb-action to fix Code tests

Merge #10896

10896: hir: resolve assoc trait type in path r=jhgg a=jhgg

fixes #9802

- [ ] write tests, maybe, if this is even a good fix...

Co-authored-by: Jake Heinz <jh@discordapp.com>

add test

Try to fix Code tests

simplify??

hir: resolve assoc trait type

fix: emit trait names in moniker identifier

Merge #10872

10872: ide_db: build symbol index from crate def map r=Veykril a=jhgg

fixes #4842, #10764

Is this looking correct? :eyes:

- [x] build the symbol index based upon the CrateDefMap for the given crate in `crate_symbols`
   - [x] make it multi threaded again, and figure out how to cache each moduleid's symbol index in salsa.
   - [x] NavigationTarget for names in macros is wrong, need to figure out how to compute a text range in the original file id?
   - [x] cleanup some duped code
   - [x] collect macros from `ItemScope.declared_macros()` into symbol index.
        - [x] store declared macros in `ItemScope` so we can figure out where macros were defined for the index.
   - [x] do something about `SymbolIndex::for_files` - ideally it should use the new module symbol index stuff.
       - [x] delete `source_file_to_file_symbols` & co...
           - [x] figure out what to do about `library_symbols`
           - [x] maybe... speed up the new `library_symbols` - the new impl is probably much slower, and definitely much less parallel. **deciding to do nothing here, we can optimize later if necerssary.**
   - [x] fix failing test: `navigation_target::tests::test_nav_for_symbol` - notably the crate def map doesn't seem to find declarations inside function.
       - [x] now a bunch of other tests are failing around auto_import & qualify_path handlers. :(
           - [x] need to assoc items in traits and impls

Co-authored-by: Jake Heinz <jh@discordapp.com>

simpler way of grabbing module /  trait name

remove one need for semantics

consts with inners??

pr feedbacks

Merge #10887

10887: internal: Update bug report template to include version info r=Veykril a=jonas-schievink

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

Update bug report template to include version info

Merge #10871

10871: Respect `http.proxyStrictSSL` r=lnicola a=lnicola

Closes #10866

Currently untested.

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

Merge #10841

10841: Emit moniker in lsif r=Veykril a=HKalbasi

fix #10559

Co-authored-by: hkalbasi <hamidrezakalbasi@protonmail.com>
Co-authored-by: HKalbasi <45197576+HKalbasi@users.noreply.github.com>

test the one thing this pr was trying to accomplish lol...

add a test

some more cleanups

do something with library_symbols

fix tests

remove duplicate data from CrateOrigin

simplify work

:broom:

collect macro_rules! macros into macro declarations

collect macros

cleanup the whole thing...