aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | vscode: add error handling to downloadFile()Veetaha2020-02-111-3/+10
| | |/ / | |/| |
* | | | Merge #3092bors[bot]2020-02-111-2/+1
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3092: vscode: fix binary is not functional on windows r=matklad a=Veetaha This is my first approach to fix this error, need to double-check this on windows still... Fixes #3087 Fixes #3090 Co-authored-by: Veetaha <[email protected]>
| * | | | vscode: fix ebusy error on windowsVeetaha2020-02-101-2/+1
| |/ / /
* | | | Merge #3102bors[bot]2020-02-113-11/+17
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | 3102: Better error messages while deserializing r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | | Better error messages while deserializingAleksey Kladov2020-02-113-11/+17
|/ / /
* | | Merge #3074bors[bot]2020-02-1021-113/+427
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3074: Or patterns r=matthewjasper a=matthewjasper Works towards #2458 Co-authored-by: Matthew Jasper <[email protected]>
| * | | Address review commentsMatthew Jasper2020-02-103-14/+66
| | | |
| * | | Update tests for or-patternsMatthew Jasper2020-02-094-60/+184
| | | |
| * | | Add or- and parenthesized-patternsMatthew Jasper2020-02-0915-46/+184
| | | |
* | | | Merge #3084bors[bot]2020-02-101-1/+2
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | 3084: More specific error if `rustc --print sysroot` fails r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | | More specific error if `rustc --print sysroot` failsAleksey Kladov2020-02-101-1/+2
|/ / /
* | | eamcs is now upstreamedAleksey Kladov2020-02-101-3/+0
| | |
* | | Merge pull request #3081 from matklad/with-outputAleksey Kladov2020-02-103-2/+48
|\ \ \ | | | | | | | | xtask release
| * | | xtask releaseAleksey Kladov2020-02-103-2/+48
| | | |
* | | | Merge #3080bors[bot]2020-02-103-19/+16
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3080: More convenient run_with_output r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | | More convenient run_with_outputAleksey Kladov2020-02-103-19/+16
|/ / /
* | | Merge #3079bors[bot]2020-02-101-0/+11
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3079: Add note r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | | Add noteAleksey Kladov2020-02-101-0/+11
| | | |
* | | | Merge #3075bors[bot]2020-02-103-6/+6
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | 3075: Migrate u32::from(TextUnit) as usize to .to_usize() r=matklad a=Veetaha @matklad I see 29 uses of `.to_usize()` cast, wouldn't it be reasonable to change `TextUnit(u32)` to `TextUnit(usize)` ? Or do you have reasons to stick with `u32` here? Co-authored-by: Veetaha <[email protected]>
| * | | move to to_usize()Veetaha2020-02-093-6/+6
| | | |
* | | | Merge #3077bors[bot]2020-02-101-4/+4
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | 3077: Move build prerequisites into build section r=matklad a=dvermd Co-authored-by: dvermd <[email protected]>
| * | | Move build prerequisites into build sectiondvermd2020-02-101-4/+4
|/ / /
* | | Merge #3072bors[bot]2020-02-091-5/+11
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3072: vscode: removed unnecessary awaits feature r=matklad a=Veetaha Found a feature that when the user has no internet connection the whole extension is blocked by waiting for the user to dismiss the error message and for making a sanity-check dns resolution. Co-authored-by: Veetaha <[email protected]>
| * | | vscode: changed dns sanity-check url from google.com to example.comVeetaha2020-02-091-3/+6
| | | |
| * | | vscode: add on-success logic to dns resolution sanity checkVeetaha2020-02-091-4/+7
| | | |
| * | | vscode: removed unnecessary awaitsVeetaha2020-02-091-2/+2
| |/ /
* | | Merge #3071bors[bot]2020-02-092-26/+63
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | | | | | | | 3071: Freshen docs for prebuilt binaries and raLspServer setting r=matklad a=Veetaha Better documented breaking changes as per an incident in #2988 Follow up for #3053 Co-authored-by: Veetaha <[email protected]>
| * | docs: quick fix inner link in docsVeetaha2020-02-091-1/+2
| | |
| * | docs: more documentation on prebuilt binaries all-editors-wiseVeetaha2020-02-091-22/+40
| | |
| * | docs: Freshen docs for prebuilt binaries and raLspServer settingVeetaha2020-02-092-5/+23
|/ /
* | Merge #3068bors[bot]2020-02-091-13/+37
|\ \ | | | | | | | | | | | | | | | | | | | | | 3068: Do not import anything if first segment of the qualified path resolves r=matklad a=SomeoneToIgnore Part of initial https://github.com/rust-analyzer/rust-analyzer/pull/3061, closing 2nd issue mentioned in the last comment there. Co-authored-by: Kirill Bulatov <[email protected]>
| * | Fix rebase leftoversKirill Bulatov2020-02-091-2/+2
| | |
| * | Do not import anything if first segment of FQN resolvesKirill Bulatov2020-02-091-12/+36
| | |
* | | Merge #3053bors[bot]2020-02-0911-30/+429
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3053: Feature: downloading lsp server from GitHub r=matklad a=Veetaha This is currently very WIP, I may need to change this and that, add "download language server command", logging stuff (for future bug reports), etc., but it already works. Also didn't test this on windows yet and mac (don't have the latter) The quirks: * Downloaded binary doesn't have executable permissions by default, that's why we ~~`chmod 111`~~ (**[UPD]** `chmod 755` as per @lnicola [suggestion](https://github.com/rust-analyzer/rust-analyzer/pull/3053#discussion_r376694456)) for it. * To remove installed binary run `rm /${HOME}/.config/Code/User/globalStorage/matklad.rust-analyzer/ra_lsp_server-linux`, ~~note that `-f` flag is necessary, because of `111` permissions (I think this should be changed)~~ (**[UPD]** --force is no longer needed due to 755 permissions). I also tried to keep things simple and not to use too many dependencies, all the ones added have 0 dependencies, (`ts-not-nil` is my personal npm package, that imitates `unwrap()` in TypeScript) **[UPD]** I reduced throttle latency of progress indicator to 200ms for smoother UX // TODO: - [x] ~~Add `Rust Analyzer: Download latest language server` vscode command.~~ **[UPD]**: having reviewed the code and estimated available options I concluded that this feature requires too many code changes, I'd like to extract this into a separate PR after we merge this one. - [x] Add some logging for future debugging - [x] ~~Gracefully handle the case when language server is not available (e.g. no internet connection, user explicitly rejected the download, etc.)~~ **[UPD]** Decided to postpone better implementation of graceful degradation logic as per [conversation](https://rust-lang.zulipchat.com/#narrow/stream/185405-t-compiler.2Fwg-rls-2.2E0/topic/Deployment.20and.20installation/near/187758550). Demo (**[UPD]** this is a bit outdated, but still mainly reflects the feature): ![ra-github-release-download-mvp](https://user-images.githubusercontent.com/36276403/74077961-4f248a80-4a2d-11ea-962f-27c650fd6c4c.gif) Related issue: #2988 #3007 Co-authored-by: Veetaha <[email protected]> Co-authored-by: Veetaha <[email protected]>
| * | docs: fix spelling untill -> untilVeetaha2020-02-091-1/+1
| | | | | | | | | Co-Authored-By: Jonas Platte <[email protected]>
| * | docs: minor fixes in wordings and punctuationVeetaha2020-02-091-2/+2
| | |
| * | vscode: updated docs on prebuilt binariesVeetaha2020-02-091-0/+32
| | |
| * | vscode: added logging when donloading binariesVeetaha2020-02-093-5/+26
| | |
| * | vscode: move throtting of download progress to call siteVeetaha2020-02-092-16/+19
| | |
| * | vscode: replaced unwrapNotNil() with ! as per @matkladVeetaha2020-02-093-21/+3
| | |
| * | vscode: refactor inverted ternaries to if statements as per @matkladVeetaha2020-02-092-2/+6
| | |
| * | vscode: remove noise data fields declarations as per @matkladVeetaha2020-02-091-6/+1
| | |
| * | vscode: refactor levels of code nesting and string literals quotesVeetaha2020-02-091-26/+30
| | |
| * | vscode: fix chmod to octal literalVeetaha2020-02-091-1/+1
| | | | | | | | | Co-Authored-By: Aleksey Kladov <[email protected]>
| * | vscode: minor names and message contents changesVeetaha2020-02-082-9/+12
| | |
| * | vscode: rename raLspServer variable to langServerVeetaha2020-02-081-3/+3
| | |
| * | vscode: refactor commentVeetaha2020-02-081-1/+1
| | |
| * | vscode: refactor platform artifact name query to switch statement, move ↵Veetaha2020-02-084-70/+89
| | | | | | | | | | | | BinarySource union variants into a namespace
| * | vscode: reduce throttle latency of downloadFile() progress callback for ↵Veetaha2020-02-081-1/+1
| | | | | | | | | | | | smoother UX
| * | vscode: add name to the second unused argument of withProgress() callbackVeetaha2020-02-081-1/+1
| | |