2019-03-20 12:22:05 +00:00
# Contributing Quick Start
2020-06-16 14:44:56 +00:00
Rust Analyzer is an ordinary Rust project, which is organized as a Cargo
2019-03-20 12:22:05 +00:00
workspace, builds on stable and doesn't depend on C libraries. So, just
```
$ cargo test
```
should be enough to get you started!
2021-02-02 18:59:27 +00:00
To learn more about how rust-analyzer works, see [./architecture.md ](./architecture.md ) document.
It also explains the high-level layout of the source code.
Do skim through that document.
2019-03-20 12:22:05 +00:00
2019-03-25 07:27:43 +00:00
We also publish rustdoc docs to pages:
2020-08-13 15:42:52 +00:00
https://rust-analyzer.github.io/rust-analyzer/ide/
2019-03-25 07:27:43 +00:00
Various organizational and process issues are discussed in this document.
2019-03-20 12:22:05 +00:00
# Getting in Touch
Rust Analyzer is a part of [RLS-2.0 working
group](https://github.com/rust-lang/compiler-team/tree/6a769c13656c0a6959ebc09e7b1f7c09b86fb9c0/working-groups/rls-2.0).
Discussion happens in this Zulip stream:
https://rust-lang.zulipchat.com/#narrow/stream/185405-t-compiler.2Fwg-rls-2.2E0
# Issue Labels
* [good-first-issue ](https://github.com/rust-analyzer/rust-analyzer/labels/good%20first%20issue )
are good issues to get into the project.
2020-06-02 09:16:49 +00:00
* [E-has-instructions ](https://github.com/rust-analyzer/rust-analyzer/issues?q=is%3Aopen+is%3Aissue+label%3AE-has-instructions )
2019-03-20 12:22:05 +00:00
issues have links to the code in question and tests.
* [E-easy ](https://github.com/rust-analyzer/rust-analyzer/issues?q=is%3Aopen+is%3Aissue+label%3AE-easy ),
[E-medium ](https://github.com/rust-analyzer/rust-analyzer/issues?q=is%3Aopen+is%3Aissue+label%3AE-medium ),
[E-hard ](https://github.com/rust-analyzer/rust-analyzer/issues?q=is%3Aopen+is%3Aissue+label%3AE-hard ),
2020-10-14 13:20:31 +00:00
[E-unknown ](https://github.com/rust-analyzer/rust-analyzer/issues?q=is%3Aopen+is%3Aissue+label%3AE-unknown ),
labels are *estimates* for how hard would be to write a fix. Each triaged issue should have one of these labels.
* [S-actionable ](https://github.com/rust-analyzer/rust-analyzer/issues?q=is%3Aopen+is%3Aissue+label%3AS-actionable ) and
[S-unactionable ](https://github.com/rust-analyzer/rust-analyzer/issues?q=is%3Aopen+is%3Aissue+label%3AS-unactionable )
specify if there are concrete steps to resolve or advance an issue. Roughly, actionable issues need only work to be fixed,
while unactionable ones are effectively wont-fix. Each triaged issue should have one of these labels.
2019-04-04 10:51:43 +00:00
* [fun ](https://github.com/rust-analyzer/rust-analyzer/issues?q=is%3Aopen+is%3Aissue+label%3Afun )
2019-03-20 12:22:05 +00:00
is for cool, but probably hard stuff.
2021-01-26 11:18:28 +00:00
* [Design ](https://github.com/rust-analyzer/rust-analyzer/issues?q=is%3Aopen+is%3Aissue+label%Design )
is for moderate/large scale architecture discussion.
Also a kind of fun.
These issues should generally include a link to a Zulip discussion thread.
2019-03-20 12:22:05 +00:00
2019-03-20 12:34:09 +00:00
# CI
2020-01-29 13:45:32 +00:00
We use GitHub Actions for CI. Most of the things, including formatting, are checked by
2019-03-20 12:34:09 +00:00
`cargo test` so, if `cargo test` passes locally, that's a good sign that CI will
2020-02-08 15:02:37 +00:00
be green as well. The only exception is that some long-running tests are skipped locally by default.
2020-01-29 13:45:32 +00:00
Use `env RUN_SLOW_TESTS=1 cargo test` to run the full suite.
We use bors-ng to enforce the [not rocket science ](https://graydon2.dreamwidth.org/1597.html ) rule.
2019-03-20 13:05:49 +00:00
# Launching rust-analyzer
2020-08-02 12:37:27 +00:00
Debugging the language server can be tricky.
LSP is rather chatty, so driving it from the command line is not really feasible, driving it via VS Code requires interacting with two processes.
2019-03-20 13:05:49 +00:00
2020-08-02 12:37:27 +00:00
For this reason, the best way to see how rust-analyzer works is to find a relevant test and execute it.
VS Code & Emacs include an action for running a single test.
2019-03-20 13:05:49 +00:00
2020-08-02 12:37:27 +00:00
Launching a VS Code instance with a locally built language server is also possible.
There's ** "Run Extension (Debug Build)"** launch configuration for this in VS Code.
2020-01-29 13:45:32 +00:00
2020-08-02 12:37:27 +00:00
In general, I use one of the following workflows for fixing bugs and implementing features:
2020-01-29 13:45:32 +00:00
2020-08-02 12:37:27 +00:00
If the problem concerns only internal parts of rust-analyzer (i.e. I don't need to touch the `rust-analyzer` crate or TypeScript code), there is a unit-test for it.
So, I use **Rust Analyzer: Run** action in VS Code to run this single test, and then just do printf-driven development/debugging.
As a sanity check after I'm done, I use `cargo xtask install --server` and **Reload Window** action in VS Code to verify that the thing works as I expect.
2020-01-29 13:45:32 +00:00
2020-08-02 12:37:27 +00:00
If the problem concerns only the VS Code extension, I use **Run Installed Extension** launch configuration from `launch.json` .
Notably, this uses the usual `rust-analyzer` binary from `PATH` .
For this, it is important to have the following in your `settings.json` file:
2020-02-09 18:19:23 +00:00
```json
{
2021-01-07 15:06:11 +00:00
"rust-analyzer.server.path": "rust-analyzer"
2020-02-09 18:19:23 +00:00
}
```
2020-10-27 07:48:20 +00:00
After I am done with the fix, I use `cargo xtask install --client` to try the new extension for real.
2020-06-02 09:16:49 +00:00
2020-08-02 12:37:27 +00:00
If I need to fix something in the `rust-analyzer` crate, I feel sad because it's on the boundary between the two processes, and working there is slow.
I usually just `cargo xtask install --server` and poke changes from my live environment.
Note that this uses `--release` , which is usually faster overall, because loading stdlib into debug version of rust-analyzer takes a lot of time.
To speed things up, sometimes I open a temporary hello-world project which has `"rust-analyzer.withSysroot": false` in `.code/settings.json` .
This flag causes rust-analyzer to skip loading the sysroot, which greatly reduces the amount of things rust-analyzer needs to do, and makes printf's more useful.
Note that you should only use the `eprint!` family of macros for debugging: stdout is used for LSP communication, and `print!` would break it.
2020-06-02 09:16:49 +00:00
2020-08-02 12:37:27 +00:00
If I need to fix something simultaneously in the server and in the client, I feel even more sad.
I don't have a specific workflow for this case.
2020-06-02 09:16:49 +00:00
2020-08-02 12:37:27 +00:00
Additionally, I use `cargo run --release -p rust-analyzer -- analysis-stats path/to/some/rust/crate` to run a batch analysis.
This is primarily useful for performance optimizations, or for bug minimization.
2020-06-15 08:33:17 +00:00
2020-08-02 12:37:27 +00:00
## TypeScript Tests
2020-06-13 12:27:23 +00:00
2020-08-02 12:37:27 +00:00
If you change files under `editors/code` and would like to run the tests and linter, install npm and run:
2020-06-13 12:27:23 +00:00
2020-08-02 12:37:27 +00:00
```bash
cd editors/code
npm ci
npm run lint
2020-06-08 10:54:48 +00:00
```
2020-08-02 12:37:27 +00:00
# Code Style & Review Process
2020-07-24 23:35:33 +00:00
2020-08-02 12:37:27 +00:00
Do see [./style.md ](./style.md ).
2020-07-24 23:35:33 +00:00
2021-02-05 14:44:04 +00:00
# How to ...
* ... add an assist? [#7535 ](https://github.com/rust-analyzer/rust-analyzer/pull/7535 )
* ... add a new protocol extension? [#4569 ](https://github.com/rust-analyzer/rust-analyzer/pull/4569 )
* ... add a new configuration option? [#7451 ](https://github.com/rust-analyzer/rust-analyzer/pull/7451 )
* ... add a new completion? [#6964 ](https://github.com/rust-analyzer/rust-analyzer/pull/6964 )
* ... allow new syntax in the parser? [#7338 ](https://github.com/rust-analyzer/rust-analyzer/pull/7338 )
2019-03-20 13:05:49 +00:00
# Logging
Logging is done by both rust-analyzer and VS Code, so it might be tricky to
figure out where logs go.
Inside rust-analyzer, we use the standard `log` crate for logging, and
2019-11-30 00:36:48 +00:00
`env_logger` for logging frontend. By default, log goes to stderr, but the
stderr itself is processed by VS Code.
2019-03-20 13:05:49 +00:00
To see stderr in the running VS Code instance, go to the "Output" tab of the
panel and select `rust-analyzer` . This shows `eprintln!` as well. Note that
`stdout` is used for the actual protocol, so `println!` will break things.
To log all communication between the server and the client, there are two choices:
* you can log on the server side, by running something like
```
2021-01-25 11:14:31 +00:00
env RA_LOG=lsp_server=debug code .
2019-03-20 13:05:49 +00:00
```
* you can log on the client side, by enabling `"rust-analyzer.trace.server":
"verbose"` workspace setting. These logs are shown in a separate tab in the
output and could be used with LSP inspector. Kudos to
[@DJMcNab ](https://github.com/DJMcNab ) for setting this awesome infra up!
2020-06-16 14:44:56 +00:00
There are also two VS Code commands which might be of interest:
2019-03-20 13:05:49 +00:00
2020-07-22 11:40:45 +00:00
* `Rust Analyzer: Status` shows some memory-usage statistics.
2019-03-20 13:05:49 +00:00
* `Rust Analyzer: Syntax Tree` shows syntax tree of the current file/selection.
2019-04-14 20:04:08 +00:00
2021-01-01 19:25:18 +00:00
* `Rust Analyzer: View Hir` shows the HIR expressions within the function containing the cursor.
2020-04-02 08:23:56 +00:00
You can hover over syntax nodes in the opened text file to see the appropriate
rust code that it refers to and the rust editor will also highlight the proper
text range.
2020-06-08 15:50:27 +00:00
If you trigger Go to Definition in the inspected Rust source file,
the syntax tree read-only editor should scroll to and select the
2020-04-02 08:23:56 +00:00
appropriate syntax node token.
2019-04-14 20:04:08 +00:00
# Profiling
2020-02-17 20:05:48 +00:00
We have a built-in hierarchical profiler, you can enable it by using `RA_PROFILE` env-var:
2019-04-14 20:04:08 +00:00
```
RA_PROFILE=* // dump everything
RA_PROFILE=foo|bar|baz // enabled only selected entries
2019-04-14 20:18:58 +00:00
RA_PROFILE=*@3>10 // dump everything, up to depth 3, if it takes more than 10 ms
2019-04-14 20:04:08 +00:00
```
2019-04-14 20:18:58 +00:00
2020-02-17 20:05:48 +00:00
In particular, I have `export RA_PROFILE='*>10'` in my shell profile.
2019-06-16 16:19:38 +00:00
2021-01-21 16:04:50 +00:00
We also have a "counting" profiler which counts number of instances of popular structs.
It is enabled by `RA_COUNT=1` .
2019-06-16 16:19:38 +00:00
To measure time for from-scratch analysis, use something like this:
```
2020-02-18 11:33:16 +00:00
$ cargo run --release -p rust-analyzer -- analysis-stats ../chalk/
2019-06-16 16:19:38 +00:00
```
For measuring time of incremental analysis, use either of these:
```
2020-02-18 11:33:16 +00:00
$ cargo run --release -p rust-analyzer -- analysis-bench ../chalk/ --highlight ../chalk/chalk-engine/src/logic.rs
$ cargo run --release -p rust-analyzer -- analysis-bench ../chalk/ --complete ../chalk/chalk-engine/src/logic.rs:94:0
2019-06-16 16:19:38 +00:00
```
2020-08-05 11:50:09 +00:00
# Release Process
Release process is handled by `release` , `dist` and `promote` xtasks, `release` being the main one.
2020-08-05 12:01:51 +00:00
`release` assumes that you have checkouts of `rust-analyzer` , `rust-analyzer.github.io` , and `rust-lang/rust` in the same directory:
2020-08-05 11:50:09 +00:00
```
./rust-analyzer
./rust-analyzer.github.io
./rust-rust-analyzer # Note the name!
```
Additionally, it assumes that remote for `rust-analyzer` is called `upstream` (I use `origin` to point to my fork).
Release steps:
1. Inside rust-analyzer, run `cargo xtask release` . This will:
* checkout the `release` branch
* reset it to `upstream/nightly`
* push it to `upstream` . This triggers GitHub Actions which:
2020-08-10 23:01:25 +00:00
* runs `cargo xtask dist` to package binaries and VS Code extension
* makes a GitHub release
* pushes VS Code extension to the marketplace
2020-08-05 11:50:09 +00:00
* create new changelog in `rust-analyzer.github.io`
2021-02-12 17:52:51 +00:00
2. While the release is in progress, fill in the changelog
2020-08-05 11:50:09 +00:00
3. Commit & push the changelog
4. Tweet
5. Inside `rust-analyzer` , run `cargo xtask promote` -- this will create a PR to rust-lang/rust updating rust-analyzer's submodule.
Self-approve the PR.
2020-12-16 15:03:15 +00:00
2021-02-01 14:43:18 +00:00
If the GitHub Actions release fails because of a transient problem like a timeout, you can re-run the job from the Actions console.
If it fails because of something that needs to be fixed, remove the release tag (if needed), fix the problem, then start over.
Make sure to remove the new changelog post created when running `cargo xtask release` a second time.
2021-03-22 12:08:04 +00:00
We release "nightly" every night automatically and promote the latest nightly to "stable" manually, every week.
2021-03-22 11:59:56 +00:00
We don't do "patch" releases, unless something truly egregious comes up.
2020-12-16 15:03:15 +00:00
# Permissions
There are three sets of people with extra permissions:
* rust-analyzer GitHub organization **admins** (which include current t-compiler leads).
Admins have full access to the org.
* **review** team in the organization.
Reviewers have `r+` access to all of organization's repositories and publish rights on crates.io.
They also have direct commit access, but all changes should via bors queue.
It's ok to self-approve if you think you know what you are doing!
bors should automatically sync the permissions.
2020-12-16 15:07:21 +00:00
* **triage** team in the organization.
2020-12-16 15:03:15 +00:00
This team can label and close issues.
