aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* minorAleksey Kladov2020-04-231-4/+0
|
* Fully get rid of SyntaxNodePtr::rangeAleksey Kladov2020-04-233-24/+40
|
* Merge #4102bors[bot]2020-04-233-11/+8
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 <[email protected]>
| * Cleanup proc_macro configAleksey Kladov2020-04-233-11/+8
| | | | | | | | | | | | | | 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 #4100bors[bot]2020-04-231-1/+1
|\ \ | |/ |/| | | | | | | | | | | 4100: Bump required rust to 1.43 r=matklad a=kjeremy This only impacts source builds Co-authored-by: kjeremy <[email protected]>
| * Bump required rust to 1.43kjeremy2020-04-231-1/+1
| |
* | Merge #4065bors[bot]2020-04-232-7/+217
|\ \ | |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | 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 <[email protected]>
| * Update tests to reflect new completionsnathanwhit2020-04-221-1/+1
| |
| * Add tests for enum completionnathanwhit2020-04-221-0/+178
| | | | | | | | Adds tests for completion of enum variants in match arms, if-let statements, and basic expressions.
| * Complete unqualified enum variants when possiblenathanwhit2020-04-221-5/+26
| |
| * Add utility fn for expected type of a nodenathanwhit2020-04-221-1/+12
| | | | | | | | | | Adds `expected_type_of` to `CompletionContext` to return the expected type of a node, if it is known.
* | Merge #4099bors[bot]2020-04-233-3/+8
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4099: Remove SyntaxPtr::range from more places r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | Remove SyntaxPtr::range from more placesAleksey Kladov2020-04-233-3/+8
|/ /
* | Merge #4093bors[bot]2020-04-231-1/+1
|\ \ | | | | | | | | | | | | | | | | | | | | | 4093: Simplify config r=matklad a=Veetaha Co-authored-by: veetaha <[email protected]>
| * | Simplify configveetaha2020-04-221-1/+1
| | |
* | | Merge #4094bors[bot]2020-04-231-1/+1
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4094: proc_macro: add ability to log to stderr and view output in vscode r=matklad a=Veetaha r? @edwin0cheng Co-authored-by: veetaha <[email protected]>
| * | | proc_macro: add ability to log to stderr and view output in vscodeveetaha2020-04-221-1/+1
| |/ /
* | | Merge #4092bors[bot]2020-04-233-4/+38
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 <[email protected]>
| * | | feat: run ignored testsHannes De Valkeneer2020-04-223-4/+38
| |/ /
* | | Merge #4098bors[bot]2020-04-231-0/+20
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 <[email protected]>
| * | | Add YouCompleteMe as a LSP option for vim/neovimWeihang Lo2020-04-231-0/+20
|/ / /
* | | Merge #4096bors[bot]2020-04-231-36/+67
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 <[email protected]>
| * | | tasks.json SupportJonathan Dickinson2020-04-231-36/+67
| |/ / | | | | | | | | | | | | | | | | | | | | | 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.
* | | Merge #4095bors[bot]2020-04-231-1/+1
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | 4095: Fix typo r=edwin0cheng a=Veetaha Co-authored-by: veetaha <[email protected]>
| * | Fix typoveetaha2020-04-231-1/+1
|/ /
* | Merge #4090bors[bot]2020-04-222-3/+3
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | 4090: Fix config naming r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | Align the name of proc-macro enabling flagAleksey Kladov2020-04-222-2/+2
| | | | | | | | | | | | We use `enable`, not `enabled` elsewhere
| * | Match implementation of diagnostics flag to the docsAleksey Kladov2020-04-221-1/+1
| | |
* | | Merge #4089bors[bot]2020-04-222-50/+109
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4089: npm update r=kjeremy a=kjeremy Co-authored-by: kjeremy <[email protected]>
| * | | npm updatekjeremy2020-04-222-50/+109
| | | |
* | | | Merge #4088bors[bot]2020-04-221-8/+8
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | 4088: cargo update r=kjeremy a=kjeremy Co-authored-by: kjeremy <[email protected]>
| * | | cargo updatekjeremy2020-04-221-8/+8
|/ / /
* | | Merge #4082bors[bot]2020-04-221-8/+16
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 <[email protected]>
| * | | Don't panic on `rust-analyzer --help`Andrew Chin2020-04-221-8/+16
| |/ / | | | | | | | | | | | | | | | Special case handling for `-h` and `--help` if no subcommand is given. Closes #4068
* | | Merge #4081bors[bot]2020-04-221-2/+35
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4081: Work around crlf in syntax tree r=matklad a=Veetaha Workarounds fixes #4067 Co-authored-by: veetaha <[email protected]>
| * | | Refactor the workaround a bitveetaha2020-04-221-19/+16
| | | |
| * | | Work around crlf in syntax treeveetaha2020-04-221-2/+38
| |/ /
* | | Merge #4080bors[bot]2020-04-221-1/+1
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | 4080: Fix .rast tmGrammar to account for numbers in identifiers r=matklad a=Veetaha Co-authored-by: veetaha <[email protected]>
| * | Fix .rast tmGrammar to account for numbers in identifiersveetaha2020-04-221-1/+1
| | |
* | | Merge #4078bors[bot]2020-04-212-5/+47
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4078: Do not add default and closure types in 'add explicit type' assist r=matklad a=SomeoneToIgnore Co-authored-by: Kirill Bulatov <[email protected]>
| * | | Do not add default and closure types in 'add explicit type' assistKirill Bulatov2020-04-212-5/+47
|/ / /
* | | Merge #4076bors[bot]2020-04-213-10/+98
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 <[email protected]>
| * | | Improve remove dervie attrsEdwin Cheng2020-04-213-10/+98
| |/ /
* | | Merge #4038bors[bot]2020-04-212-1582/+1592
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 <[email protected]>
| * | | Group generated ast boilerplate apart from the interesting partveetaha2020-04-182-1582/+1592
| | | |
* | | | Merge #4071bors[bot]2020-04-211-7/+9
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4071: Cleanup args a bit r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | Cleanup args a bitAleksey Kladov2020-04-211-7/+9
| | | | |
* | | | | Merge #4070bors[bot]2020-04-211-2/+2
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4070: docs(readme): improve user docs r=matklad a=fannheyward cleanup unavailable configurations/commands Co-authored-by: Heyward Fann <[email protected]>
| * | | | | docs(readme): improve user docsHeyward Fann2020-04-211-2/+2
|/ / / / / | | | | | | | | | | | | | | | cleanup unavailable configurations/commands
* | | | | Merge #4069bors[bot]2020-04-211-14/+25
|\ \ \ \ \ | |_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 <[email protected]>