Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Test inner and outer doc comments in hover | Lukas Wirth | 2020-12-09 | 1 | -0/+62 |
| | |||||
* | Properly fetch inner and outer docs on hir-level | Lukas Wirth | 2020-12-09 | 2 | -17/+33 |
| | |||||
* | Properly fetch inner and outer attributes on hir-level | Lukas Wirth | 2020-12-08 | 1 | -5/+39 |
| | |||||
* | Merge #6760 | bors[bot] | 2020-12-08 | 2 | -15/+17 |
|\ | | | | | | | | | | | | | | | | | 6760: Open Cargo.toml opens more specific manifest r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]> | ||||
| * | Open Cargo.toml opens more specific manifest | Aleksey Kladov | 2020-12-08 | 2 | -10/+12 |
| | | |||||
| * | cleanup: match over unwrap | Aleksey Kladov | 2020-12-08 | 1 | -5/+5 |
| | | |||||
* | | Merge #6750 | bors[bot] | 2020-12-08 | 12 | -213/+137 |
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6750: Remove documentation query, move doc handling to attributes r=matklad a=Veykril Fixes #3182 Removes the documentation query in favor of `Attrs::docs`. Attrs already handlded doc comments partially but the alloc saving check was wrong so it only worked when other attributes existed as well. Unfortunately the `new` constructor has to do an intermediate allocation now because we need to keep the order of mixed doc attributes and doc comments. I've also partially adjusted the `hover` module to have its tests check the changes, it still has some `HasSource` trait usage due to the `ShortLabel` trait usage, as that is only implemented on the Ast parts and not the Hir, should this ideally be implemented for the Hir types as well?(would be a follow up PR of course) Co-authored-by: Lukas Wirth <[email protected]> | ||||
| * | | Simplify ast::Comment api surface | Lukas Wirth | 2020-12-08 | 1 | -10/+15 |
| | | | |||||
| * | | Escape string literals in Attr::from_src | Lukas Wirth | 2020-12-08 | 2 | -12/+6 |
| | | | |||||
| * | | Replace Arc<[str]> with String in attr::Documentation | Lukas Wirth | 2020-12-07 | 1 | -2/+2 |
| | | | |||||
| * | | Keep doc attribute order | Lukas Wirth | 2020-12-07 | 3 | -55/+61 |
| | | | |||||
| * | | Remove raw pre and suffixes from string attr literals | Lukas Wirth | 2020-12-07 | 1 | -2/+5 |
| | | | |||||
| * | | Remove hir_def/docs.rs module | Lukas Wirth | 2020-12-07 | 6 | -114/+63 |
| | | | |||||
| * | | Remove documentation query | Lukas Wirth | 2020-12-07 | 6 | -59/+26 |
| | | | |||||
* | | | Tweak the search query params for better lookup speed | Kirill Bulatov | 2020-12-08 | 2 | -32/+38 |
| | | | |||||
* | | | Better config name | Kirill Bulatov | 2020-12-08 | 5 | -13/+13 |
| | | | |||||
* | | | Disable the completion for no corresponding client resolve capabilities | Kirill Bulatov | 2020-12-07 | 11 | -160/+84 |
| | | | |||||
* | | | Document the feature | Kirill Bulatov | 2020-12-07 | 1 | -0/+43 |
| | | | |||||
* | | | Simplify | Kirill Bulatov | 2020-12-07 | 4 | -36/+35 |
| | | | |||||
* | | | Check lsp completions' edits for disjointness | Kirill Bulatov | 2020-12-07 | 2 | -1/+176 |
| | | | |||||
* | | | Less panic, more tests | Kirill Bulatov | 2020-12-07 | 3 | -11/+62 |
| | | | |||||
* | | | Fix the profiling label | Kirill Bulatov | 2020-12-07 | 1 | -1/+1 |
| | | | |||||
* | | | Use stateless completion resolve | Kirill Bulatov | 2020-12-07 | 6 | -71/+124 |
| | | | |||||
* | | | Remove the state | Kirill Bulatov | 2020-12-07 | 7 | -107/+37 |
| | | | |||||
* | | | Make completion resolve async | Kirill Bulatov | 2020-12-07 | 7 | -53/+116 |
| | | | |||||
* | | | Simplify import edit calculation | Kirill Bulatov | 2020-12-07 | 12 | -111/+114 |
| | | | |||||
* | | | Remove redundant code | Kirill Bulatov | 2020-12-07 | 8 | -103/+80 |
| | | | |||||
* | | | Refactor the code | Kirill Bulatov | 2020-12-07 | 11 | -34/+48 |
| | | | |||||
* | | | Add eager resolve capability | Kirill Bulatov | 2020-12-07 | 8 | -13/+43 |
| | | | |||||
* | | | Better support client completion resolve caps | Kirill Bulatov | 2020-12-07 | 5 | -105/+120 |
| | | | |||||
* | | | Properly fill client completion resolve capabilities data | Kirill Bulatov | 2020-12-07 | 5 | -5/+56 |
| | | | |||||
* | | | Resolve import inserts better | Kirill Bulatov | 2020-12-07 | 2 | -38/+91 |
| | | | |||||
* | | | Remove unnecessary leftovers | Kirill Bulatov | 2020-12-07 | 2 | -3/+0 |
| | | | |||||
* | | | Working resolve completion imports prototype | Kirill Bulatov | 2020-12-07 | 9 | -46/+78 |
| | | | |||||
* | | | Draft the new lsp handler | Kirill Bulatov | 2020-12-07 | 5 | -24/+52 |
| |/ |/| | |||||
* | | Apply environment set by build scripts | Jonas Schievink | 2020-12-07 | 2 | -2/+20 |
| | | |||||
* | | Merge #6752 | bors[bot] | 2020-12-07 | 1 | -0/+5 |
|\ \ | |/ |/| | | | | | | | | | | | 6752: Pass `--target` when loading out dirs from check r=jonas-schievink a=jonas-schievink bors r+ Co-authored-by: Jonas Schievink <[email protected]> | ||||
| * | Pass `--target` when loading out dirs from check | Jonas Schievink | 2020-12-07 | 1 | -0/+5 |
| | | |||||
* | | Merge #6719 | bors[bot] | 2020-12-07 | 5 | -23/+23 |
|\ \ | |/ |/| | | | | | | | | | | | 6719: Use items can also have doc comments r=matklad a=Veykril Prior to this change modules show more docs than they have cause they inherit the docs from documented use items inside of them. Co-authored-by: Lukas Wirth <[email protected]> | ||||
| * | Use items can also have doc comments | Lukas Wirth | 2020-12-04 | 5 | -23/+23 |
| | | |||||
* | | Use the right `def_crate` for builtin macros | Jonas Schievink | 2020-12-07 | 1 | -1/+1 |
| | | |||||
* | | Merge #6745 | bors[bot] | 2020-12-07 | 11 | -99/+83 |
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6745: Some more proc macro cleanups r=jonas-schievink a=jonas-schievink * Remove `ProcMacroClient::dummy` and just use `Option<ProcMacroClient>` instead * Remember the type of proc macros (later allows us to reject using an incorrect macro type) * Prepare a few internals for procedural attribute macros bors r+ Co-authored-by: Jonas Schievink <[email protected]> | ||||
| * | | Remove resolved FIXME | Jonas Schievink | 2020-12-07 | 1 | -1/+0 |
| | | | |||||
| * | | Remove dummy ProcMacroClient in favor of Option | Jonas Schievink | 2020-12-07 | 5 | -71/+59 |
| | | | |||||
| * | | Retain types of proc macros and allow attr. macros | Jonas Schievink | 2020-12-07 | 5 | -22/+29 |
| | | | |||||
| * | | Make `ProcMacroProcessExpander` private | Jonas Schievink | 2020-12-07 | 1 | -1/+1 |
| | | | |||||
| * | | ProcMacroProcessExpander: support attribute macros | Jonas Schievink | 2020-12-07 | 2 | -21/+11 |
| | | | |||||
* | | | Merge #6743 | bors[bot] | 2020-12-07 | 2 | -13/+11 |
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6743: Don't insert blank lines between doc attributes r=Veykril a=Veykril Fixes #6742. Doc attributes should be concatenated via a single linebreak as written in the [rustdoc book](https://doc.rust-lang.org/nightly/rustdoc/the-doc-attribute.html). Also changed the loop to use an iterator to get rid of the `docs.trim_end_matches("\n\n").to_owned()` part using `Itertools::intersperse`. Co-authored-by: Lukas Wirth <[email protected]> | ||||
| * | | | Don't insert blank lines between doc attributes | Lukas Wirth | 2020-12-07 | 2 | -13/+11 |
| | | | | |||||
* | | | | Merge #6731 | bors[bot] | 2020-12-07 | 7 | -5/+319 |
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6731: Add replace_match_with_if_let assist r=matklad a=Veykril Basically the counterpart to `replace_if_let_with_match`, I personally sometimes want to replace matches like ```rust match foo { pat => expr, _ => (), } ``` into the corresponding ```rust if let pat = foo { expr } ``` which is the main reasoning behind this. I put this into the same file as `replace_if_let_with_match` because the are complementing each other and I would probably rename the file to something like `replace_if_let_match` but I didn't do that for now because I was unsure whether git would still view this as a rename or not due to the amount of changes in the file so that the diff is still properly visible for now. Co-authored-by: Lukas Wirth <[email protected]> |