aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | Merge #1097bors[bot]2019-04-031-2/+5
|\ \ \ \ \ | |_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1097: Fix VSCode cargo-watch functionality on Linux. r=matklad a=kiljacken As of #1079 the VSCode cargo-watch functionality has been broken on Linux systems. The cause seems to be that linux takes the added quotes inside process arguments literally, so it attempts to make cargo-watch run the command `cargo "check --message-format json"` with the entire quoted part being treated as a single long subcommand, which cargo doesn't know how to handle. Removing the extra quotes solves the issue. Closes #1096 Co-authored-by: Emil Lauridsen <[email protected]>
| * | | | Fix eslint errorsEmil Lauridsen2019-04-031-1/+1
| | | | |
| * | | | Add extra double quotes only on Windows.Emil Lauridsen2019-04-031-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As tested by @edwin0cheng, Windows requires the quotes removed in the previous commit. This commit re-adds the quotes gated by an if statement on the node environment, so that quotes are only added on Windows.
| * | | | Fix VSCode cargo-watch functionality on Linux.Emil Lauridsen2019-04-031-2/+1
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As of #1079 the VSCode cargo-watch functionality has been broken on Linux systems. The cause seems to be that linux takes the added quotes inside process arguments literally, so it attempts to make cargo-watch run the command `cargo "check --message-format json"` with the entire quoted part being treated as a single long subcommand, which cargo doesn't know how to handle. Removing the extra quotes solves the issue.
* | | | Merge #1076bors[bot]2019-04-0210-49/+231
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1076: Const body inference r=flodiebold a=Lapz This is the second part of #887. I've added type inference on const bodies and introduced the DefWithBody containing Function, Const and Static. I want to add tests but im unsure on how I would go about testing that completions work. Co-authored-by: Lenard Pratt <[email protected]>
| * | | | made ExprCollector pub(crate) and movedLenard Pratt2019-04-022-14/+11
| | | | | | | | | | | | | | | | | | | | collect_fn_body_syntax
| * | | | Added const bodies and static body to the astLenard Pratt2019-04-0210-91/+180
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | and added inference the inference test reduce code duplication
| * | | | Added defWithBodyLenard Pratt2019-03-307-24/+120
| | | | |
* | | | | Merge #1094bors[bot]2019-04-022-44/+78
|\ \ \ \ \ | |_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1094: Improve cargo-watch animation and refactoring r=matklad a=edwin0cheng * Refactoring code and fixed some minor bugs. * Add more error log and better error handling. * Add current checking artifact name in Cargo-watch animation. ![new_status_animation](https://i.imgur.com/Zr2JLa5.gif) Co-authored-by: Edwin Cheng <[email protected]>
| * | | | Add cargo-watch package animation and refactoringEdwin Cheng2019-04-022-44/+78
|/ / / /
* | | | Merge #1093bors[bot]2019-04-027-400/+336
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1093: simplify r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | simplifyAleksey Kladov2019-04-027-400/+336
| | | | |
* | | | | Merge #1092bors[bot]2019-04-026-29/+70
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1092: always produce source for import r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | always produce source for importAleksey Kladov2019-04-026-29/+70
|/ / / /
* | | | Merge #1090bors[bot]2019-04-02221-8921/+8921
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1090: always show token text r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | always show token textAleksey Kladov2019-04-02221-8921/+8921
| | | | |
* | | | | Merge #1091bors[bot]2019-04-021-1/+1
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1091: fix inner block doc comments r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | fix inner block doc commentsAleksey Kladov2019-04-021-1/+1
|/ / / /
* | | | Merge #1089bors[bot]2019-04-025-23/+26
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1089: fix a panic with glob-import missing a source map r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | fix a panic with glob-import missing a source mapAleksey Kladov2019-04-025-23/+26
| | | | |
* | | | | Merge #1079bors[bot]2019-04-0211-44/+448
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1079: Improve cargo-watch usage in vscode plugin r=matklad a=edwin0cheng *This PR try to improve current cargo-watch usage in VSCode :* 1. Add Multi-lines error support : ![multilines-error](https://i.imgur.com/gbLEwMG.gif) 2. Add cargo-watch status animation : ![cargo-watch-status](https://i.imgur.com/GbHwzjj.gif) *Implementation Details* * Current VSCode `ProblemMatcher` still do not support multiple line parsing. * However we can, spawn a cargo watch process instead of using vscode.Task to allow more control. * Use `cargo-check --message-format json` to get json format of compiler-message. * Use `vscode.DiagnosticCollection` to manage the problems directly, which allow multiple lines diagnostic. However, * VSCode use non mono-space font for problems, at this moment i cannot find a good solution about it. * I am not so good in typescript, please let me know if anything is bad in this PR. Co-authored-by: Edwin Cheng <[email protected]> Co-authored-by: Edwin Cheng <[email protected]>
| * | | | | Fix prettier errorEdwin Cheng2019-04-021-1/+1
| | | | | |
| * | | | | Add terminate process implemntation noteEdwin Cheng2019-04-021-0/+7
| | | | | |
| * | | | | Add Cargo.toml file check before cargo watch startEdwin Cheng2019-04-021-0/+22
| | | | | |
| * | | | | Add cargo-watch.check-argumentsEdwin Cheng2019-04-026-74/+139
| | | | | |
| * | | | | Add config for cargo-watch traceEdwin Cheng2019-04-025-15/+67
| | | | | |
| * | | | | Add proper process teminate methodEdwin Cheng2019-04-026-10/+116
| | | | | |
| * | | | | Fix prettier errorEdwin Cheng2019-04-024-22/+31
| | | | | |
| * | | | | Fixed tslint errorEdwin Cheng2019-04-021-1/+1
| | | | | |
| * | | | | Fix tslint errorEdwin Cheng2019-04-022-63/+65
| | | | | |
| * | | | | Improve cargo-watch usageEdwin Cheng2019-04-024-36/+177
| | | | | |
* | | | | | Merge #1088bors[bot]2019-04-029-582/+612
|\ \ \ \ \ \ | | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1088: add minimal comments r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | | add minimal commentsAleksey Kladov2019-04-025-0/+13
| | | | | |
| * | | | | rename flavor to kindAleksey Kladov2019-04-026-53/+53
| | | | | |
| * | | | | move extensions to submodulesAleksey Kladov2019-04-023-549/+566
| | | | | |
| * | | | | remove flavorAleksey Kladov2019-04-023-10/+10
| | | | | |
* | | | | | Merge #1087bors[bot]2019-04-024-36/+58
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1087: More future-proof comment kind r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | | More future-proof comment kindAleksey Kladov2019-04-024-36/+58
|/ / / / /
* | | | | Merge #1085bors[bot]2019-04-0211-276/+292
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1085: add ast::tokens r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | fix comment namingAleksey Kladov2019-04-022-13/+15
| | | | |
| * | | | allow empty doc commentsAleksey Kladov2019-04-021-4/+6
| | | | |
| * | | | add ast::tokensAleksey Kladov2019-04-0210-119/+124
| | | | |
| * | | | simplifyAleksey Kladov2019-04-021-23/+19
| | | | |
| * | | | move ast traits to a separate fileAleksey Kladov2019-04-022-144/+155
|/ / / /
* | | | Merge #1084bors[bot]2019-04-013-37/+0
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1084: remove dead code r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | remove dead codeAleksey Kladov2019-04-013-37/+0
| | | | |
* | | | | Merge #1050bors[bot]2019-04-015-45/+402
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1050: Intelligently add parens when inlining local varaibles r=matklad a=gfreezy fixed this. https://github.com/rust-analyzer/rust-analyzer/pull/1037#discussion_r268627141 Co-authored-by: gfreezy <[email protected]>
| * | | | intelligently add parens when inlining local varaiblesgfreezy2019-04-015-45/+402
|/ / / /
* | | | Merge #1082bors[bot]2019-04-013-0/+96
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1082: Async block in argument position r=matklad a=andreytkachenko Fixes case when async block appears in argument position Co-authored-by: Andrey Tkachenko <[email protected]>
| * | | | Async block in argument positionAndrey Tkachenko2019-04-013-0/+96
| | | | |