Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
| * | Fixed #5129 | petr-tik | 2020-07-27 | 1 | -8/+23 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Addresses two issues: - keep the parens from dbg!() in case the call is chained or there is semantic difference if parens are excluded - Exclude the semicolon after the dbg!(); by checking if it was accidentally included in the macro_call investigated, but decided against: fix ast::MacroCall extraction to never include semicolons at the end - this logic lives in rowan. Defensively shorten the macro_range if there is a semicolon token. Deleted unneccessary temp variable macro_args Renamed macro_content to "paste_instead_of_dbg", because it isn't a simple extraction of text inside dbg!() anymore | |||||
| * | Make all test fn names consistent in remove_dbg | petr-tik | 2020-07-27 | 1 | -3/+29 | |
| | | ||||||
| * | Added failing tests | petr-tik | 2020-07-27 | 1 | -0/+21 | |
| | | ||||||
* | | Rename NomialDef -> AdtDef | Aleksey Kladov | 2020-07-29 | 2 | -3/+3 | |
|/ | ||||||
* | assists: change_return_type_to_result: clarify assist description | Matthias Krüger | 2020-07-22 | 1 | -1/+1 | |
| | | | | | | | | I had a -> Option<PathBuf> fn, which I wanted to change to Result<PathBuf, _>, but despite advertising to do so, the assist did not change the result type to Result<PathBuf, _> but instead just wrapped it in a Result: <Result<Option<PathBuf>, _>. I changed the assist description to "Wrap return type in Result" to clarify that the assist only wraps the preexisting type and does not do any deep Option-to-Result refactoring. | |||||
* | Replace existing visibility modifier in fix_visibility | Timo Freiberg | 2020-07-21 | 1 | -15/+75 | |
| | ||||||
* | Cleanup extact variable | Aleksey Kladov | 2020-07-21 | 1 | -48/+55 | |
| | ||||||
* | minor | Aleksey Kladov | 2020-07-21 | 1 | -2/+2 | |
| | ||||||
* | Minor perf tweaks per clippy | Jeremy Kolb | 2020-07-19 | 2 | -2/+2 | |
| | ||||||
* | Add turbo-fish works after `()` | Aleksey Kladov | 2020-07-18 | 1 | -1/+28 | |
| | ||||||
* | More precise ranges in remove hashes assist | Aleksey Kladov | 2020-07-17 | 1 | -79/+52 | |
| | ||||||
* | Merge #5350 | bors[bot] | 2020-07-15 | 4 | -5/+98 | |
|\ | | | | | | | | | | | | | | | | | | | | | 5350: Filter assists r=matklad a=kjeremy Uses the `CodeActionContext::only` field to compute only those assists the client cares about. It works but I don't really like the implementation. Co-authored-by: kjeremy <[email protected]> Co-authored-by: Jeremy Kolb <[email protected]> | |||||
| * | Move allow list into AssistConfig | Jeremy Kolb | 2020-07-15 | 4 | -26/+24 | |
| | | ||||||
| * | Filter assists | kjeremy | 2020-07-13 | 3 | -13/+108 | |
| | | ||||||
* | | Move remove_bounds to edit.rs | Jonas Schievink | 2020-07-14 | 1 | -9/+1 | |
| | | ||||||
* | | missing impl members: remove assoc. type bounds | Jonas Schievink | 2020-07-14 | 1 | -0/+33 | |
| | | ||||||
* | | Add a license field to all the crates | Yuki Okushi | 2020-07-14 | 1 | -0/+1 | |
|/ | ||||||
* | change vis works on statics | Aleksey Kladov | 2020-07-12 | 1 | -3/+13 | |
| | ||||||
* | Better caret placement when filling match arms | Aleksey Kladov | 2020-07-10 | 1 | -124/+86 | |
| | ||||||
* | Don't mess with cursor position when adding hashes | Aleksey Kladov | 2020-07-09 | 1 | -9/+22 | |
| | ||||||
* | Refactor AssistBuilder to manage a SourceChange | Christoph Herzog | 2020-07-07 | 1 | -7/+7 | |
| | | | | | | | `AssistBuilder`` now managaes a full `SourceChange` instead of a Vec<SourceFileEdit>. This prepares AssistBuilder to handle creation of new files. | |||||
* | Add AssistKind::Generate | Aleksey Kladov | 2020-07-03 | 6 | -25/+31 | |
| | ||||||
* | Unify naming of generating assists | Aleksey Kladov | 2020-07-03 | 7 | -206/+215 | |
| | ||||||
* | Don't categorize things we don't care about | kjeremy | 2020-07-02 | 4 | -64/+52 | |
| | ||||||
* | Move AssistKind into AssistId | kjeremy | 2020-07-02 | 40 | -139/+89 | |
| | ||||||
* | Recategorize a few | kjeremy | 2020-07-02 | 3 | -3/+3 | |
| | ||||||
* | Categorize assists | Jeremy Kolb | 2020-07-02 | 40 | -501/+702 | |
| | ||||||
* | Record and suggest trait items via ImportMap | Jonas Schievink | 2020-07-02 | 1 | -0/+140 | |
| | ||||||
* | Reuse Semantics instances | Laurențiu Nicola | 2020-07-01 | 2 | -2/+2 | |
| | ||||||
* | Use the existing Semantics in auto_import | Laurențiu Nicola | 2020-07-01 | 1 | -6/+6 | |
| | ||||||
* | Remove db from AssistsContext | Laurențiu Nicola | 2020-07-01 | 11 | -35/+37 | |
| | ||||||
* | Make SemanticsScope non-generic | Laurențiu Nicola | 2020-07-01 | 1 | -10/+6 | |
| | ||||||
* | Use more of FxHash* | Laurențiu Nicola | 2020-06-29 | 1 | -3/+3 | |
| | ||||||
* | Get rid of unused imports | Veetaha | 2020-06-28 | 2 | -10/+3 | |
| | ||||||
* | Follow matklad suggestions | Veetaha | 2020-06-28 | 4 | -18/+13 | |
| | | | | | | - Move vis_offset() to utils.rs - Shorten explicit ra_syntax::ast -> ast - Undo refactoring exhaustive pattern to non-exhaustive | |||||
* | Simlify with matches!() | Veetaha | 2020-06-28 | 3 | -17/+5 | |
| | ||||||
* | introduce_variable -> extract_variable | Aleksey Kladov | 2020-06-27 | 3 | -73/+73 | |
| | ||||||
* | Merge #4945 | bors[bot] | 2020-06-26 | 1 | -2/+31 | |
|\ | | | | | | | | | | | | | | | 4945: do not suggest assist for return type to result in bad case r=matklad a=bnjjj close #4826 Co-authored-by: Benjamin Coenen <[email protected]> | |||||
| * | do not suggest assist for return type to result in bad case #4826 | Benjamin Coenen | 2020-06-21 | 1 | -0/+3 | |
| | | | | | | | | Signed-off-by: Benjamin Coenen <[email protected]> | |||||
| * | do not suggest assist for return type to result in bad case #4826 | Benjamin Coenen | 2020-06-18 | 1 | -2/+28 | |
| | | | | | | | | Signed-off-by: Benjamin Coenen <[email protected]> | |||||
* | | Smarter introduce variable | Aleksey Kladov | 2020-06-25 | 1 | -8/+49 | |
| | | | | | | | | Use field init shorthand | |||||
* | | Use only one code-path for parsing fixtures | Aleksey Kladov | 2020-06-23 | 1 | -6/+7 | |
| | | | | | | | | | | This removes leading newlines everywhere, shifting all ranges in tests by one | |||||
* | | Centralize fixture parsing for assists | Aleksey Kladov | 2020-06-23 | 6 | -54/+47 | |
| | | ||||||
* | | More principled indentation trimming in fixtures | Aleksey Kladov | 2020-06-23 | 3 | -123/+127 | |
| | | ||||||
* | | New VFS | Aleksey Kladov | 2020-06-23 | 1 | -10/+3 | |
| | | ||||||
* | | Fix panic in split and merge import assists | Laurențiu Nicola | 2020-06-23 | 2 | -1/+21 | |
| | | ||||||
* | | Remove RelativePathBuf from fixture | Aleksey Kladov | 2020-06-22 | 3 | -3/+3 | |
|/ | | | | | The paths in fixture are not really relative (the default one is `/main.rs`), so it doesn't make sense to use `RelativePathBuf` here. | |||||
* | Merge #4878 | bors[bot] | 2020-06-16 | 4 | -59/+249 | |
|\ | | | | | | | | | | | | | | | 4878: Make "Replace qualified name with use" replace *all* mentions of the path r=matklad a=jonas-schievink Fixes https://github.com/rust-analyzer/rust-analyzer/issues/4836 Co-authored-by: Jonas Schievink <[email protected]> | |||||
| * | Operate only on AST paths instead of HIR | Jonas Schievink | 2020-06-15 | 1 | -30/+36 | |
| | | ||||||
| * | Simplify | Jonas Schievink | 2020-06-15 | 1 | -13/+6 | |
| | |