Merge #5873

5873: Correct sed replacement when called in a bash file r=matklad a=pksunkara

There's so much trial and error regarding shells and stuff I needed to do to get this correctly working on github runner.

Co-authored-by: Pavan Kumar Sunkara <pavan.sss1991@gmail.com>

Merge #5874

5874: Remove AttrDef
 r=matklad a=matklad

bors r+
🤖

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

Remove AttrDef

Correct sed replacement when called in a bash file

Merge #5870

5870: Move code_model attrs to a new module r=matklad a=matklad

bors r+
🤖

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

Simplify

Move attrs from code_module to a separate module

Merge #5869

5869: Minor
 r=matklad a=matklad

bors r+
🤖

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

Minor

Merge #5867

5867: Use the same abstraction for attrs and docs
 r=matklad a=matklad

bors r+
🤖

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

Use the same abstraction for attrs and docs

Doc comments *are* attributes, so there's no reason to have two crates
here.

Merge #5866

5866: Improve logging
 r=matklad a=matklad

bors r+
🤖

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

Improve logging

Merge #5865

5865: Rename rust_analyzer lib in the binaries before publishing r=matklad a=pksunkara

@matklad Tested and working with publish dry run. Please run the action manually once merged.

Co-authored-by: Pavan Kumar Sunkara <pavan.sss1991@gmail.com>

Rename rust_analyzer lib in the binaries before publishing

Merge #4873

4873: Resolve links in hover documentation r=matklad a=zacps

This PR resolves links in hover documentation. Both the upcoming intra-doc-links style and the old "path-based" style.

## Todo

* [x] More tests
* [ ] Benchmark (Is there an easy way to benchmark this?)
* [x] ~~Resolve issues with the markdown parser/get rid of it~~ Migrate to `pulldown_cmark_to_cmark`
* [x] Reorganise code (Tips appreciated)

---

Fixes #503

Co-authored-by: Zac Pullar-Strecker <zacmps@gmail.com>

Merge #5863

5863: Cleanup invert-if
 r=matklad a=matklad

bors r+
🤖

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

Cleanup invert-if

* stick to trivial factory functions in make
* compress the logic for inverting Option/Result

Merge remote-tracking branch 'upstream/master' into 503-hover-doc-links

Changes from review

Merge #5861

5861: Support extern types r=matklad a=jonas-schievink

This is a currently unstable feature tracked at https://github.com/rust-lang/rust/issues/43467

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

Support extern types

Merge #5859

5859: Allow rust-analyzer auto publishing r=matklad a=pksunkara

Co-authored-by: Pavan Kumar Sunkara <pavan.sss1991@gmail.com>

Allow rust-analyzer auto publishing

Merge #5841

5841: Gate stream.pipeline workaround on fixed versions of node r=matklad a=Veetaha

Fixes the symptom of https://github.com/cdr/code-server/issues/1810
Original report here: https://github.com/rust-analyzer/rust-analyzer/issues/3167#issuecomment-678390564
Thanks to @hjfreyer for precise investigation :D

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

Merge #5733

5733: Fix expand glob import bugs r=matklad a=unexge

fixes https://github.com/rust-analyzer/rust-analyzer/issues/5709

TODOs:
- [x] Incorrect node replacing
<details>
    <summary>Details</summary>

```rust
use crate::{
    body::scope::{ExprScopes, ScopeId},
    body::Body,
    builtin_type::BuiltinType,
    db::DefDatabase,
    expr::{ExprId, PatId},
    generics::GenericParams,
    item_scope::{BuiltinShadowMode, BUILTIN_SCOPE},
    nameres::CrateDefMap,
    path::*<|>,
    per_ns::PerNs,
    visibility::{RawVisibility, Visibility},
    AdtId, AssocContainerId, ConstId, ContainerId, DefWithBodyId, EnumId, EnumVariantId,
    FunctionId, GenericDefId, HasModule, ImplId, LocalModuleId, Lookup, ModuleDefId, ModuleId,
    StaticId, StructId, TraitId, TypeAliasId, TypeParamId, VariantId,
};
```
becames
```rust
use crate::{PathKind, name, name, ModPath};
```
</details>

- [x] Ignoring visibility
<details>
    <summary>Details</summary>

```rust
mod foo {
    mod bar {
        pub struct Bar;
    }
}

use foo::bar::*;

fn baz(bar: Bar) {}
```
becames
```rust
mod foo {
    mod bar {
        pub struct Bar;
    }
}

use foo::bar::Bar;

fn baz(bar: Bar) {}
```
although mod `bar` is private
</details>

- [x] Eating attributes

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

Merge #5855

5855: Add TBD description to arena r=matklad a=pksunkara

Sorry I missed this, @matklad. Please run manually again once this is merged.

Co-authored-by: Pavan Kumar Sunkara <pavan.sss1991@gmail.com>

Add TBD description to arena

Merge #4776

4776: Do a weekly minor publish to crates.io r=matklad a=pksunkara

This is the same system I set up on Chalk repo.

Every week it creates a new minor version, pushes it to github and then deploys it to crates.io.

Co-authored-by: Pavan Kumar Sunkara <pavan.sss1991@gmail.com>

Merge #5845

5845: Omit lenses for not runnable doctests r=matklad a=SomeoneToIgnore

Ideally, we should properly parse the doctest attributes before, but since I need it for the code lens only, this way should suffice for now

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

Add description for crates that will be published

Added weekly publish workflow

Merge #5851

5851: Invert if should be smart about is_some, is_none, is_ok, is_err r=matklad a=dragfire

Fixes #5827

Co-authored-by: dragfire <asem.devajit@gmail.com>

Merge #5852

5852: Add Early Return rule to style r=matklad a=matklad

bors r+

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

Merge #5853

5853: Avoid monomorphization
 r=matklad a=matklad

bors r+
🤖

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

Avoid monomorphization

Add Early Return rule to style

Renames, comments, and dead code removal

Merge remote-tracking branch 'upstream/master' into 503-hover-doc-links

Add version to deps in cargo.toml

Add a test to document unimplemented case

Reorganise code

Invert if should be smart about is_some, is_none, is_ok, is_err

Omit lenses for not runnable doctests

Merge #5823

5823: Don't underline function definition if self is &mut r=Nashenas88 a=matklad

The self is right there, and is already underlined, so it makes little
sense to emit even more underlines.

before:
after:

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

Merge #5839

5839: Dont ask me why... r=kjeremy a=Veetaha

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

Gate stream.pipeline workaround on fixed versions of node

Dont ask me why...

Merge #5838

5838: :arrow_up: autocfg
 r=matklad a=matklad

bors r+
🤖

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

:arrow_up: autocfg

:arrow_up: ungrammar

Merge #5835

5835: Switch to expect_test from crates.io
 r=matklad a=matklad

bors r+
🤖

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

Switch to expect_test from crates.io

Remove expect crate

Use new `Definition::usages` API in expand glob import

Fix importing private modules in expand glob import

Fix importing unused traits in expand glob import

Use fixme instead of todo

Use `Definition::find_usages` for finding used items in expand glob import

Run rustfmt

Handle more cases in AST replacing in expand glob import

Improve AST replacing in expand glob import

Merge #5832

5832: Update crates r=kjeremy a=kjeremy

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

Update crates

Merge #5824

5824: Optimize reference search
 r=matklad a=matklad

bors r+
🤖

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

Optimize reference search

Merge #5813

5813: Try lld r=matklad a=matklad

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

Use lld on winsows

Don't underline function definition if self is &mut

The self is right there, and is already underlined, so it makes little
sense to emit even more underlines.

Merge #5822

5822: Remove dead code
 r=matklad a=matklad

bors r+
🤖

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

Remove dead code

Merge #5821

5821: **Remove Unused Parameter** refactoring
 r=matklad a=matklad

bors r+
🤖

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

**Remove Unused Parameter** refactoring

Merge #5820

5820: Future proof find-usages API
 r=matklad a=matklad

bors r+
🤖

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

Future proof find-usages API

We might want to provide more efficient impls for check if usages
exist, limiting the search, filtering and cancellation, so let's
violate YAGNI a bit here.

Merge #5819

5819: Apply couple of rule of thumbs to simplify highlighting code
 r=matklad a=matklad

bors r+
🤖

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

Apply couple of rule of thumbs to simplify highlighting code

Main one: instead of adding a parameter to function to handle special
case, make the caller handle it.

Second main one: make sure that function does a reasonable thing.
`highlight_def` picks  a color for def, *regardless* of the context
the def is use. Feeding an info from the call-site muddies the
responsibilities here.

Minor smells, flagging the function as having space for improvement in
the first place:

* many parameters, some of which are set as constants on most
call-sites (introduce severalfunction instad)
* boolean param (add two functions instead)

Merge #5818

5818: Some trivial local simiplifications r=matklad a=matklad

bors r+
🤖

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

Inline trivial function

Simplify

Minor

Minor cleanups

Better name

Merge #5817

5817: Better API factoring around self access modes
 r=matklad a=matklad

bors r+
🤖

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

Better API factoring around self access modes

Merge #5816

5816: Better error if Cargo is not in Path
 r=matklad a=matklad

bors r+
🤖

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

Better error if Cargo is not in Path

Merge #5815

5815: Introduce hir::Param
 r=matklad a=matklad

bors r+
🤖

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

Introduce hir::Param

We generally shouldn't expose TypeRef out of hir. So, let's just use a
placehoder here.

Merge #5814

5814: Add SelfParam to code_model
 r=matklad a=matklad

bors r+
🤖

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

Add SelfParam to code_model

Merge #5812

5812: Minor cleanups
 r=matklad a=matklad

bors r+
🤖

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

Minor cleanups

Merge #5811

5811: Minor
 r=matklad a=matklad

bors r+
🤖

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

Minor

Merge #5643

5643: Add new consuming modifier, apply consuming and mutable to methods r=matklad a=Nashenas88

This adds a new `consuming` semantic modifier for syntax highlighters.

This also emits `mutable` and `consuming` in two cases:

- When a method takes `&mut self`, then it now has `function.mutable` emitted.
- When a method takes `self`, and the type of `Self` is not `Copy`, then `function.consuming` is emitted.

CC @flodiebold

Co-authored-by: Paul Daniel Faria <Nashenas88@users.noreply.github.com>

Merge #5809

5809: :arrow_up: crates
 r=matklad a=matklad

bors r+
🤖

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

:arrow_up: crates

Merge #5808

5808: Make RacyFlag actually work
 r=matklad a=matklad

bors r+
🤖

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

Make RacyFlag actually work