Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Rename UseItem -> Use | Aleksey Kladov | 2020-07-30 | 4 | -8/+8 |
| | |||||
* | Rename Rename | Aleksey Kladov | 2020-07-30 | 1 | -1/+1 |
| | |||||
* | Split ItemList & AssocItemList | Aleksey Kladov | 2020-07-30 | 4 | -5/+7 |
| | |||||
* | Merge #5554 | bors[bot] | 2020-07-29 | 1 | -9/+85 |
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 5554: Fix remove_dbg r=matklad a=petr-tik Closes #5129 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 Co-authored-by: petr-tik <[email protected]> | ||||
| * | Addressed code review | petr-tik | 2020-07-29 | 1 | -27/+41 |
| | | | | | | | | | | | | | | replaced match with let-if variable assignment removed the unnecessary semicolon_on_end variable converted all code and expected test variables to raw strings and inlined them in asserts | ||||
| * | 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 |
|/ | |||||
* | 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 |
| | |