aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* vscode: remove chmod in favour of an option to createWriteStream()Veetaha2020-02-112-5/+8
|
* Merge #3112bors[bot]2020-02-111-28/+57
|\ | | | | | | | | | | | | | | 3112: Return early, return often r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * Return early, return oftenAleksey Kladov2020-02-111-27/+33
| |
| * Fix join lines when two rules matchAleksey Kladov2020-02-111-23/+46
|/
* Merge #3064bors[bot]2020-02-112-25/+68
|\ | | | | | | | | | | | | | | 3064: Handle macro token cases for rename r=matklad a=edwin0cheng Fixes #2957 Co-authored-by: Edwin Cheng <[email protected]>
| * Use get_or_insert_withEdwin Cheng2020-02-091-4/+3
| |
| * Handle macro token cases for renameEdwin Cheng2020-02-092-25/+69
| |
* | Third time's a charm?Aleksey Kladov2020-02-111-1/+1
| |
* | Fix releaseAleksey Kladov2020-02-111-9/+0
| |
* | Merge pull request #3086 from matklad/auto-publish-extensionAleksey Kladov2020-02-111-3/+19
|\ \ | | | | | | Publish VS Code extension automatically
| * | Publish VS Code extension automaticallyAleksey Kladov2020-02-111-3/+19
| | |
| | |
| \ \
*-. \ \ Merge #3083 #3101bors[bot]2020-02-112-12/+26
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3083: Update some crates r=matklad a=kjeremy 3101: vscode: filter out arm linux from using prebuilt binaries r=matklad a=Veetaha Closes #3076 Co-authored-by: kjeremy <[email protected]> Co-authored-by: Veetaha <[email protected]>
| | * | | vscode: filter out arm linux from using prebuilt binariesVeetaha2020-02-111-3/+20
| | | | |
| * | | | Update some crateskjeremy2020-02-101-9/+6
| | | | |
* | | | | Merge #3100bors[bot]2020-02-111-3/+10
|\ \ \ \ \ | |_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3100: vscode: add error handling to downloadFile() r=matklad a=Veetaha Inspired by #3094. Co-authored-by: Veetaha <[email protected]>
| * | | | 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]>
generated by cgit v1.2.3 (git 2.25.1) at 2024-11-29 11:30:57 +0000