aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Check that no file contains trailing wsAleksey Kladov2020-03-176-127/+142
| | | | rustfmt allows trailing spaces in string literals unfortunately.
* Merge #3625bors[bot]2020-03-171-2/+2
|\ | | | | | | | | | | | | | | 3625: Fix typo r=flodiebold a=phynalle It doesn't need any description! I just fixed a typo. haha Co-authored-by: phynalle <[email protected]>
| * Fix typophynalle2020-03-171-2/+2
| |
* | Merge pull request #3613 from bjorn3/patch-1Aleksey Kladov2020-03-171-0/+3
|\ \ | |/ |/| Fix cargo registry caching for builds
| * Fix cargo registry caching for buildsbjorn32020-03-161-0/+3
| |
* | Merge #3580bors[bot]2020-03-1714-181/+471
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3580: More error-resilient MBE expansion r=matklad a=flodiebold This is the beginning of an attempt to make macro-by-example expansion more resilient, so that we still get an expansion even if no rule exactly matches, with the goal to make completion work better in macro calls. The general idea is to make everything return `(T, Option<ExpandError>)` instead of `Result<T, ExpandError>`; and then to try each macro arm in turn, and somehow choose the 'best' matching rule if none matches without errors. Finding that 'best' match isn't done yet; I'm currently counting how many tokens were consumed from the args before an error, but it also needs to take into account whether there were further patterns that had nothing to match. I'll continue this later, but I'm interested whether you think this is the right path, @matklad & @edwin0cheng. Co-authored-by: Florian Diebold <[email protected]> Co-authored-by: Florian Diebold <[email protected]>
| * | Small fixesFlorian Diebold2020-03-161-4/+2
| | |
| * | Some more refactoringFlorian Diebold2020-03-162-58/+76
| | |
| * | Some cleanupFlorian Diebold2020-03-161-5/+7
| | |
| * | Turn ExpandResult into structFlorian Diebold2020-03-166-43/+63
| | |
| * | Fix remaining test failureFlorian Diebold2020-03-162-9/+11
| | |
| * | Fix performance problemFlorian Diebold2020-03-162-32/+37
| | |
| * | Better fix for stuck parser?Florian Diebold2020-03-161-3/+3
| | |
| * | Add test, remove printlnsFlorian Diebold2020-03-164-3/+53
| | |
| * | Get tests workingFlorian Diebold2020-03-165-7/+19
| | |
| * | wipFlorian Diebold2020-03-165-94/+160
| | |
| * | Attempt to implement ranking of rules when none matches perfectly (wip)Florian Diebold2020-03-163-11/+51
| | |
| * | Make MBE expansion more resilient (WIP)Florian Diebold2020-03-1610-91/+168
| |/
* | Merge #3624bors[bot]2020-03-173-131/+145
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3624: Tidier tidy checks r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | Tidier tidy checksAleksey Kladov2020-03-173-131/+145
|/ /
* | Merge pull request #3618 from swarnimarun/patch-1Aleksey Kladov2020-03-171-0/+1
|\ \ | | | | | | Gate release action to rust-analyzer owner
| * | Update .github/workflows/release.yamlAleksey Kladov2020-03-171-1/+1
| | | | | | | | | Co-Authored-By: Florian Diebold <[email protected]>
| * | Gate release action to rust-analyzer ownerSwarnim Arun2020-03-161-0/+1
| | |
* | | Merge #3616bors[bot]2020-03-171-16/+7
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | 3616: Update dependencies, removing rustc_version r=matklad a=kjeremy Co-authored-by: kjeremy <[email protected]>
| * | Update dependencies, removing rustc_versionkjeremy2020-03-161-16/+7
| |/
* | Merge #3614bors[bot]2020-03-167-65/+80
|\ \ | |/ |/| | | | | | | | | | | | | | | | | 3614: Separate persistent mutable state from config r=matklad a=matklad That way, we clearly see which things are not change, and we also clearly see which things are persistent. r? @Veetaha Co-authored-by: Aleksey Kladov <[email protected]>
| * Separate persistent mutable state from configAleksey Kladov2020-03-167-65/+80
| | | | | | | | | | That way, we clearly see which things are not change, and we also clearly see which things are persistent.
| * Fix audit caching betterAleksey Kladov2020-03-161-1/+2
| |
* | Merge pull request #3612 from matklad/audit5Aleksey Kladov2020-03-161-1/+2
|\ \ | |/ |/| Fix audit caching better
| * Fix audit caching betterAleksey Kladov2020-03-161-1/+2
| |
* | Merge pull request #3611 from matklad/audit4Aleksey Kladov2020-03-161-1/+2
|\| | | | | Fix cargo audit caching
| * Fix cargo audit cachingAleksey Kladov2020-03-161-1/+2
|/ | | | | | See https://github.com/actions/cache/issues/133#issuecomment-599102035 for chown bit
* Merge pull request #3610 from matklad/audit3Aleksey Kladov2020-03-161-1/+14
|\ | | | | Cache cargo-audit on CI
| * Actually make npm audit more robustAleksey Kladov2020-03-161-1/+1
| | | | | | | | In bash, && and || have the same priority.
| * Cache cargo-audit on CIAleksey Kladov2020-03-161-0/+13
| | | | | | | | closes #3399
* | Merge #3584bors[bot]2020-03-1651-794/+813
|\ \ | |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3584: Switch to dynamic dispatch r=matklad a=matklad Benches are in https://github.com/rust-analyzer/rust-analyzer/issues/1987#issuecomment-598807185 TL;DR: * 33% faster release build * slightly worse/same perf * no changes for debug build * slightly smaller binary cc @flodiebold I genuinely don't know if it is a good idea or not. Co-authored-by: Aleksey Kladov <[email protected]>
| * Use `dyn Trait` for working with databseAleksey Kladov2020-03-1651-794/+813
|/ | | | | | | It improves compile time in `--release` mode quite a bit, it doesn't really slow things down and, conceptually, it seems closer to what we want the physical architecture to look like (we don't want to monomorphise EVERYTHING in a single leaf crate).
* Merge pull request #3607 from murlakatamenka/masterAleksey Kladov2020-03-161-0/+15
|\ | | | | Add installation notes for Arch Linux users
| * User Manual: fix AUR linkmurlakatamenka2020-03-161-1/+1
| |
| * Add installation notes for Arch Linux usersmurlakatamenka2020-03-161-0/+15
| |
* | Merge pull request #3608 from matklad/audit2Aleksey Kladov2020-03-161-0/+4
|\ \ | |/ |/| Retry rust audit as well
| * Retry rust audit as wellAleksey Kladov2020-03-161-0/+4
|/ | | | https://github.com/rust-analyzer/rust-analyzer/runs/511553989?check_suite_focus=true#step:3:6
* Merge #3604bors[bot]2020-03-161-1/+1
|\ | | | | | | | | | | | | | | 3604: Remove dat fixme r=matklad a=Veetaha Cleanup as per #3603 Co-authored-by: Veetaha <[email protected]>
| * Remove dat fixmeVeetaha2020-03-161-1/+1
| |
* | Merge pull request #3606 from matklad/auditAleksey Kladov2020-03-163-23/+23
|\ \ | | | | | | Make audit more reliable
| * | Make audit more reliableAleksey Kladov2020-03-161-1/+1
| | | | | | | | | | | | | | | | | | I've seen this transient error a couple of times: https://github.com/rust-analyzer/rust-analyzer/runs/511162809?check_suite_focus=true#step:5:8
| * | :arrow_up: npmAleksey Kladov2020-03-162-22/+22
|/ /
* / Fix versionAleksey Kladov2020-03-161-1/+1
|/
* Merge #3573bors[bot]2020-03-161-0/+1
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3573: Check all crates of the workspace r=matklad a=matklad Previously, if the root of the was was a real crate, only this crate was checked. Ideally, we might want some kind of config here (which might be just overriding the whole command), but `--workspace` is def a nicer default. r? @kiljacken Co-authored-by: Aleksey Kladov <[email protected]>
| * Check all crates of the workspaceAleksey Kladov2020-03-131-0/+1
| | | | | | | | | | | | | | | | | | Previously, if the root of the was was a real crate, only this crate was checked. Ideally, we might want some kind of config here (which might be just overriding the whole command), but `--workspace` is def a nicer default.