aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* | Merge #7358bors[bot]2021-02-128-426/+460
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7358: Refactor reference searching to work with the ast r=matklad a=Veykril Addresses #4290 This PR is still a bit unpolished. Its main purpose for now is to discuss the direction of the changes as to whether this seems to be the right approach or not. I annotated a few parts with reviews to give a better overwiew without having to read into it too much. Big part of the diff are test output changes in the `references` module. Co-authored-by: Lukas Wirth <[email protected]>
| * | Update references test outputLukas Wirth2021-02-121-2/+2
| | |
| * | Use NameLike in FileReference directly as its not exported from ide anymoreLukas Wirth2021-02-125-59/+29
| | |
| * | Refactor reference searching to work with the astLukas Wirth2021-02-128-406/+489
| | |
| * | Remove unused ReferenceSearchResult functionsLukas Wirth2021-02-121-24/+5
|/ /
* | Merge #7650bors[bot]2021-02-126-18/+171
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | 7650: Add `find_impl_block_end` assist helper r=Veykril a=yoshuawuyts Fixes #7605. This makes it so assists can use helpers to either append a method to the start or the end of an `impl` block. Thanks! @Veykril if this is merged, perhaps it could be good to update the gif in https://github.com/rust-analyzer/rust-analyzer/pull/7617#issuecomment-776622135 ? -- this should fix the ordering issue when generating multiple methods. Co-authored-by: Yoshua Wuyts <[email protected]>
| * | Add `find_impl_block_end` assist helperYoshua Wuyts2021-02-126-18/+171
| |/
* | Merge #7639bors[bot]2021-02-121-1/+1
|\ \ | |/ |/| | | | | | | | | | | 7639: Bump rust to latest stable 1.50 r=kjeremy a=kjeremy Co-authored-by: kjeremy <[email protected]>
| * Bump rust to latest stable 1.50kjeremy2021-02-121-1/+1
|/
* Merge #7652bors[bot]2021-02-127-23/+52
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7652: Fix slow tests sometimes failing r=flodiebold a=flodiebold In some situations we reloaded the workspace in the tests after having reported to be ready. There's two fixes here: 1. Add a version to the VFS config and include that version in progress reports, so that we don't think we're done prematurely; 2. Delay status transitions until after changes are applied. Otherwise the last change during loading can potentially trigger a workspace reload, if it contains interesting changes. Co-authored-by: Florian Diebold <[email protected]>
| * Remove pinned Rust version againFlorian Diebold2021-02-121-9/+1
| |
| * Fix slow tests sometimes failingFlorian Diebold2021-02-126-14/+51
| | | | | | | | | | | | | | | | | | | | In some situations we reloaded the workspace in the tests after having reported to be ready. There's two fixes here: 1. Add a version to the VFS config and include that version in progress reports, so that we don't think we're done prematurely; 2. Delay status transitions until after changes are applied. Otherwise the last change during loading can potentially trigger a workspace reload, if it contains interesting changes.
* | Merge #7653bors[bot]2021-02-121-0/+60
|\ \ | |/ |/| | | | | | | | | | | | | 7653: Document config pattern r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * Document config patternAleksey Kladov2021-02-121-0/+60
| |
* | Merge #7644bors[bot]2021-02-1210-29/+89
|\ \ | | | | | | | | | | | | | | | | | | | | | 7644: Primitive completion r=jonas-schievink a=jonas-schievink Fixes https://github.com/rust-analyzer/rust-analyzer/issues/7642 Co-authored-by: Jonas Schievink <[email protected]>
| * | Complete builtin type pathsJonas Schievink2021-02-111-1/+33
| | |
| * | Wrap `BuiltinType` in code modelJonas Schievink2021-02-119-28/+56
| | |
| | |
| \ \
*-. \ \ Merge #7638 #7648bors[bot]2021-02-125-9/+9
|\ \ \ \ | |_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7638: libloading 0.7 r=kjeremy a=kjeremy See https://docs.rs/libloading/0.7.0/libloading/changelog/r0_7_0/index.html 7648: fix nightly warning `legacy_derive_helpers` r=lnicola a=peddermaster2 With a recent nightly (e.g. 2021-02-10) a warning comes up. This PR reorders the attributes to fix the warning. See https://github.com/rust-lang/rust/issues/79202 Co-authored-by: kjeremy <[email protected]> Co-authored-by: Peter Wischer <[email protected]>
| | * | fix nightly warning `legacy_derive_helpers`Peter Wischer2021-02-122-2/+2
| |/ / |/| | | | | | | | see https://github.com/rust-lang/rust/issues/79202
* | | Merge #7651bors[bot]2021-02-125-11/+21
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | 7651: Pin Rust to 1.49.0 on CI r=jonas-schievink a=jonas-schievink Co-authored-by: Jonas Schievink <[email protected]>
| * | Pin Rust to 1.49.0 on CIJonas Schievink2021-02-125-11/+21
|/ /
| * libloading 0.7kjeremy2021-02-113-7/+7
|/ | | | See https://docs.rs/libloading/0.7.0/libloading/changelog/r0_7_0/index.html
* Merge #7631bors[bot]2021-02-101-0/+29
|\ | | | | | | | | | | | | | | | | | | 7631: Add test for #1165 r=jonas-schievink a=jonas-schievink Closes #1165 bors r+ Co-authored-by: Jonas Schievink <[email protected]>
| * Add test for #1165Jonas Schievink2021-02-101-0/+29
|/ | | | Closes #1165
* Merge #7630bors[bot]2021-02-101-0/+3
|\ | | | | | | | | | | | | | | 7630: Slightly expand test r=jonas-schievink a=jonas-schievink bors r+ Co-authored-by: Jonas Schievink <[email protected]>
| * Slightly expand testJonas Schievink2021-02-101-0/+3
|/
* Merge #7627bors[bot]2021-02-105-32/+139
|\ | | | | | | | | | | | | | | 7627: infer: update resolver when descending into block r=jonas-schievink a=jonas-schievink Co-authored-by: Jonas Schievink <[email protected]>
| * Add more testsJonas Schievink2021-02-102-8/+84
| |
| * Put the old resolver backJonas Schievink2021-02-101-3/+8
| |
| * Update fixed testsJonas Schievink2021-02-103-18/+21
| |
| * infer: update resolver when descending into blockJonas Schievink2021-02-102-16/+39
|/
* Merge #7625bors[bot]2021-02-104-8/+31
|\ | | | | | | | | | | | | | | | | 7625: Add **Copy Run Command Line** command for vscode r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * Add **Copy Run Command Line** command for vscodeAleksey Kladov2021-02-104-8/+31
| | | | | | | | | | This is useful when you want to, e.g., run a specific test in a terminal with `--release`.
* | Merge #7617bors[bot]2021-02-109-68/+603
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7617: Add getter/setter assists r=Veykril a=yoshuawuyts This patch makes progress towards the design outlined in https://github.com/rust-analyzer/rust-analyzer/issues/5943, and includes a small refactor which closes https://github.com/rust-analyzer/rust-analyzer/issues/7607. All together this patch does 4 things: - Adds a `generate_getter` assist. - Adds a `generate_getter_mut` assist. - Adds a `generate_setter` assist. - Moves the `generate_impl_text` function from `generate_new` into `utils` (which closes #7607). ## Design Notes I've chosen to follow the [Rust API guidelines on getters](https://rust-lang.github.io/api-guidelines/naming.html#getter-names-follow-rust-convention-c-getter) as closely as possible. This deliberately leaves "builder pattern"-style setters out of scope. Also, similar to https://github.com/rust-analyzer/rust-analyzer/pull/7570 this assist generates doc comments. I think this should work well in most cases, and for the few where it doesn't it's probably easily edited. This makes it slightly less correct than the #7570 implementation, but I think this is still useful enough to include for many of the same reasons. The reason why this PR contains 3 assists, rather than 1, is because each of them is so similar to the others that it felt more noisy to do them separately than all at once. The amount of code added does not necessarily reflect that, but hope that still makes sense. ## Examples **Input** ```rust struct Person { name: String, // <- cursor on "name" } ``` **generate getter** ```rust struct Person { name: String, } impl Person { /// Get a reference to the person's name. fn name(&self) -> &String { &self.name } } ``` **generate mut getter** ```rust struct Person { name: String, } impl Person { /// Get a mutable reference to the person's name. fn name_mut(&mut self) -> &mut String { &mut self.name } } ``` **generate setter** ```rust struct Person { name: String, } impl Person { /// Set the person's name. fn set_name(&mut self, name: String) { self.name = name; } } ``` Co-authored-by: Yoshua Wuyts <[email protected]>
| * | Add getter/setter assistsYoshua Wuyts2021-02-099-68/+603
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Finish implementing `generate_setter` assists Make `generate_impl_text` util generic generate getter methods Fix getter / setter naming It's now in-line with the Rust API naming guidelines: https://rust-lang.github.io/api-guidelines/naming.html#getter-names-follow-rust-convention-c-getter apply clippy Improve examples
| | |
| \ \
*-. \ \ Merge #7591 #7622bors[bot]2021-02-1010-32/+67
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7591: Fix/no floating promises r=matklad a=sahandevs closes #3515 - added `@typescript-eslint/no-floating-promises: error` rule - changed `"no-console": ["error"]` to `"no-console": ["error", { allow: ["warn", "error"] }]` (we at least log the error messages of the floating promises) - fixed lint/compile errors 7622: Resolve TupleStructPat in SourceAnalyzer::resolve_path r=Veykril a=Veykril Closes #7594 bors r+ Co-authored-by: Sahandevs <[email protected]> Co-authored-by: Lukas Wirth <[email protected]>
| | * | | Resolve TupleStructPat in SourceAnalyzer::resolve_pathLukas Wirth2021-02-102-6/+34
| | | |/ | | |/|
| * | | use await insteadSahandevs2021-02-094-31/+21
| | | |
| * | | fix errorsSahandevs2021-02-074-6/+11
| | | |
| * | | formatSahandevs2021-02-074-12/+12
| | | |
| * | | handle Thenable type rejectsSahandevs2021-02-074-12/+23
| | | |
| * | | handle promise catchesSahandevs2021-02-074-4/+4
| | | |
| * | | add no-floating-promises ruleSahandevs2021-02-071-1/+2
| | | |
| * | | remove unnecessarySahandevs2021-02-074-6/+6
| | | |
* | | | Merge #7621bors[bot]2021-02-101-1/+1
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7621: Use track caller in test documentation r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | Use track caller in test documentationAleksey Kladov2021-02-101-1/+1
| | |/ / | |/| |
* | | | Merge #7619bors[bot]2021-02-101-0/+4
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7619: Add #[track_caller] to assist tests r=matklad a=yoshuawuyts This points the source of a failed assertion to the code which called it, rather than the location within the assertion helper method. While working on https://github.com/rust-analyzer/rust-analyzer/pull/7617 I had trouble locating some failing tests, and it was only by adding these attributes during development that I was able to locate them. This is only applied to test helpers, which means it comes at no runtime cost. And even then: I didn't experience any noticeable performance with this enabled or disabled. Mostly just a more pleasant experience debugging test failures (: Co-authored-by: Yoshua Wuyts <[email protected]>
| * | | Add #[track_caller] to assist testsYoshua Wuyts2021-02-091-0/+4
|/ / /
* | | Merge #7618bors[bot]2021-02-094-45/+95
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7618: Show qualified variant pattern completions r=Veykril a=Veykril ![UTMZyplssw](https://user-images.githubusercontent.com/3757771/107425157-e9c31800-6b1e-11eb-8963-96653115f8e0.gif) bors r+ Co-authored-by: Lukas Wirth <[email protected]>
| * | | Show qualified variant pattern completionsLukas Wirth2021-02-094-45/+95
|/ / /