aboutsummaryrefslogtreecommitdiff
path: root/crates/ra_assists
Commit message (Collapse)AuthorAgeFilesLines
* do not add to `pub use` statementsJacob Rothstein2020-08-032-1/+44
|
* Merge #5628bors[bot]2020-08-031-1/+1
|\ | | | | | | | | | | | | | | 5628: Rename test modules r=matklad a=lnicola Co-authored-by: Laurențiu Nicola <[email protected]>
| * Rename test modulesLaurențiu Nicola2020-07-311-1/+1
| |
* | Update grammarAleksey Kladov2020-08-011-1/+1
| |
* | Unify naming of tuple fieldsAleksey Kladov2020-07-311-2/+2
| |
* | RenameAleksey Kladov2020-07-312-6/+3
| |
* | Rename BindPat -> IdentPatAleksey Kladov2020-07-314-4/+4
| |
* | Rename PalceholderPat -> WildcardPatAleksey Kladov2020-07-313-15/+12
| |
* | Allign RecordPat with RecordExprAleksey Kladov2020-07-311-2/+2
| |
* | Fix GenericArgs grammarAleksey Kladov2020-07-311-5/+11
| |
* | Rename TypeArgList -> GenericArgListAleksey Kladov2020-07-311-2/+2
| |
* | MinorAleksey Kladov2020-07-311-1/+0
| |
* | Rename LambdaExpr -> ClosureExprAleksey Kladov2020-07-311-2/+2
| |
* | Item is a StmtAleksey Kladov2020-07-311-0/+2
| |
* | Reame PlaceholderType -> InferTypeAleksey Kladov2020-07-311-1/+1
| |
* | Rename TypeRef -> TypeAleksey Kladov2020-07-313-8/+8
|/ | | | | | | | | | The TypeRef name comes from IntelliJ days, where you often have both type *syntax* as well as *semantical* representation of types in scope. And naming both Type is confusing. In rust-analyzer however, we use ast types as `ast::Type`, and have many more semantic counterparts to ast types, so avoiding name clash here is just confusing.
* Use ty to access most TypeRefsAleksey Kladov2020-07-302-2/+2
|
* Remove TypeAscriptionOwnerAleksey Kladov2020-07-303-17/+12
|
* Finalize attribute grammarAleksey Kladov2020-07-301-2/+2
|
* Finalize impl GrammarAleksey Kladov2020-07-306-11/+11
|
* Finalize Trait grammarAleksey Kladov2020-07-302-3/+3
|
* Finalize const&static grammarAleksey Kladov2020-07-303-5/+5
|
* Rename EnumVariant -> VariantAleksey Kladov2020-07-302-3/+3
|
* Rename EnumDef -> EnumAleksey Kladov2020-07-302-3/+3
|
* Rename StructDef -> StructAleksey Kladov2020-07-303-6/+6
|
* Rename FieldDef -> FieldAleksey Kladov2020-07-302-3/+3
|
* Rename RecordLit -> RecordExprAleksey Kladov2020-07-304-6/+6
|
* Rename TypeParamList -> GenericParamListAleksey Kladov2020-07-305-13/+14
|
* Rename TypeAliasDef -> TypeAliasAleksey Kladov2020-07-303-9/+5
|
* Rename FnDef -> FnAleksey Kladov2020-07-309-17/+17
|
* Rename UseItem -> UseAleksey Kladov2020-07-304-8/+8
|
* Rename RenameAleksey Kladov2020-07-301-1/+1
|
* Split ItemList & AssocItemListAleksey Kladov2020-07-304-5/+7
|
* Merge #5554bors[bot]2020-07-291-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 reviewpetr-tik2020-07-291-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 #5129petr-tik2020-07-271-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_dbgpetr-tik2020-07-271-3/+29
| |
| * Added failing testspetr-tik2020-07-271-0/+21
| |
* | Rename NomialDef -> AdtDefAleksey Kladov2020-07-292-3/+3
|/
* assists: change_return_type_to_result: clarify assist descriptionMatthias Krüger2020-07-221-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_visibilityTimo Freiberg2020-07-211-15/+75
|
* Cleanup extact variableAleksey Kladov2020-07-211-48/+55
|
* minorAleksey Kladov2020-07-211-2/+2
|
* Minor perf tweaks per clippyJeremy Kolb2020-07-192-2/+2
|
* Add turbo-fish works after `()`Aleksey Kladov2020-07-181-1/+28
|
* More precise ranges in remove hashes assistAleksey Kladov2020-07-171-79/+52
|
* Merge #5350bors[bot]2020-07-154-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 AssistConfigJeremy Kolb2020-07-154-26/+24
| |
| * Filter assistskjeremy2020-07-133-13/+108
| |
* | Move remove_bounds to edit.rsJonas Schievink2020-07-141-9/+1
| |