diff options
author | bors[bot] <26634292+bors[bot]@users.noreply.github.com> | 2020-05-14 10:23:34 +0100 |
---|---|---|
committer | GitHub <[email protected]> | 2020-05-14 10:23:34 +0100 |
commit | 5148d6dc66d80b375a98143dfbb556ec675bbffc (patch) | |
tree | fb803ca1ef8185c0e4738d41a8f0e74c0108324a | |
parent | 6fde7f1b6bb30481a38c3346729dde9bd1b42c1a (diff) | |
parent | 9f0a7eb97b4e047cebbe51ffd6f9e2092dd63e00 (diff) |
Merge #4405
4405: Make some stuff public so that they can be reused by other tools r=pksunkara a=pksunkara
So, my little experiment of building a code analysis tool using rust-analyzer is successful. I am going to proceed to build the tool now. This PR makes the needed things public.
I know there were some things about trying to change stuff regarding loading workspaces, which would make it more easier for other tools to reuse. But, until then, it should be okay using this `load_cargo` fn.
Btw, if I were publish my tool, I would need the `ra` crates to be released. Since @matklad told me that he doesn't want to care about breaking stuff, I would propose the following.
Every monday, during the weekly release, we release a new pre v1 minor version of all the crates. That way, we don't need to care about breaking stuff but still have rust-analyzer on crates.io.
I made https://github.com/pksunkara/cargo-workspaces to help release workspace crates easily.
So, coming week, we start with `0.1.0`, then week after that, we release `0.2.0` and then `0.3.0` etc.. until we decide on `1.0.0` which is probably when the compiler team also starts using the crates. There is no limit to the minor versions (we can even have `0.150.0` or `0.1500.0`), so I don't see anything wrong with this strategy.
Co-authored-by: Pavan Kumar Sunkara <[email protected]>
-rw-r--r-- | crates/ra_ide/src/lib.rs | 8 | ||||
-rw-r--r-- | crates/ra_project_model/src/lib.rs | 2 | ||||
-rw-r--r-- | crates/ra_text_edit/src/lib.rs | 2 | ||||
-rw-r--r-- | crates/rust-analyzer/src/cli.rs | 1 | ||||
-rw-r--r-- | crates/rust-analyzer/src/cli/load_cargo.rs | 2 | ||||
-rw-r--r-- | editors/code/src/commands/ssr.ts | 2 |
6 files changed, 7 insertions, 10 deletions
diff --git a/crates/ra_ide/src/lib.rs b/crates/ra_ide/src/lib.rs index 915199bd8..78149ddfc 100644 --- a/crates/ra_ide/src/lib.rs +++ b/crates/ra_ide/src/lib.rs | |||
@@ -176,14 +176,10 @@ impl AnalysisHost { | |||
176 | pub fn request_cancellation(&mut self) { | 176 | pub fn request_cancellation(&mut self) { |
177 | self.db.request_cancellation(); | 177 | self.db.request_cancellation(); |
178 | } | 178 | } |
179 | pub fn raw_database( | 179 | pub fn raw_database(&self) -> &RootDatabase { |
180 | &self, | ||
181 | ) -> &(impl hir::db::HirDatabase + salsa::Database + ra_db::SourceDatabaseExt) { | ||
182 | &self.db | 180 | &self.db |
183 | } | 181 | } |
184 | pub fn raw_database_mut( | 182 | pub fn raw_database_mut(&mut self) -> &mut RootDatabase { |
185 | &mut self, | ||
186 | ) -> &mut (impl hir::db::HirDatabase + salsa::Database + ra_db::SourceDatabaseExt) { | ||
187 | &mut self.db | 183 | &mut self.db |
188 | } | 184 | } |
189 | } | 185 | } |
diff --git a/crates/ra_project_model/src/lib.rs b/crates/ra_project_model/src/lib.rs index 4f098b706..a2e9f65ef 100644 --- a/crates/ra_project_model/src/lib.rs +++ b/crates/ra_project_model/src/lib.rs | |||
@@ -35,7 +35,7 @@ pub enum ProjectWorkspace { | |||
35 | /// `PackageRoot` describes a package root folder. | 35 | /// `PackageRoot` describes a package root folder. |
36 | /// Which may be an external dependency, or a member of | 36 | /// Which may be an external dependency, or a member of |
37 | /// the current workspace. | 37 | /// the current workspace. |
38 | #[derive(Clone)] | 38 | #[derive(Debug, Clone)] |
39 | pub struct PackageRoot { | 39 | pub struct PackageRoot { |
40 | /// Path to the root folder | 40 | /// Path to the root folder |
41 | path: PathBuf, | 41 | path: PathBuf, |
diff --git a/crates/ra_text_edit/src/lib.rs b/crates/ra_text_edit/src/lib.rs index 37f77cc47..c4f945101 100644 --- a/crates/ra_text_edit/src/lib.rs +++ b/crates/ra_text_edit/src/lib.rs | |||
@@ -21,7 +21,7 @@ pub struct TextEdit { | |||
21 | indels: Vec<Indel>, | 21 | indels: Vec<Indel>, |
22 | } | 22 | } |
23 | 23 | ||
24 | #[derive(Debug, Default)] | 24 | #[derive(Debug, Default, Clone)] |
25 | pub struct TextEditBuilder { | 25 | pub struct TextEditBuilder { |
26 | indels: Vec<Indel>, | 26 | indels: Vec<Indel>, |
27 | } | 27 | } |
diff --git a/crates/rust-analyzer/src/cli.rs b/crates/rust-analyzer/src/cli.rs index a865a7c7e..39ce77900 100644 --- a/crates/rust-analyzer/src/cli.rs +++ b/crates/rust-analyzer/src/cli.rs | |||
@@ -16,6 +16,7 @@ use ra_syntax::{AstNode, SourceFile}; | |||
16 | pub use analysis_bench::{analysis_bench, BenchWhat, Position}; | 16 | pub use analysis_bench::{analysis_bench, BenchWhat, Position}; |
17 | pub use analysis_stats::analysis_stats; | 17 | pub use analysis_stats::analysis_stats; |
18 | pub use diagnostics::diagnostics; | 18 | pub use diagnostics::diagnostics; |
19 | pub use load_cargo::load_cargo; | ||
19 | 20 | ||
20 | #[derive(Clone, Copy)] | 21 | #[derive(Clone, Copy)] |
21 | pub enum Verbosity { | 22 | pub enum Verbosity { |
diff --git a/crates/rust-analyzer/src/cli/load_cargo.rs b/crates/rust-analyzer/src/cli/load_cargo.rs index 023ced6cf..8eaf75ff6 100644 --- a/crates/rust-analyzer/src/cli/load_cargo.rs +++ b/crates/rust-analyzer/src/cli/load_cargo.rs | |||
@@ -22,7 +22,7 @@ fn vfs_root_to_id(r: ra_vfs::VfsRoot) -> SourceRootId { | |||
22 | SourceRootId(r.0) | 22 | SourceRootId(r.0) |
23 | } | 23 | } |
24 | 24 | ||
25 | pub(crate) fn load_cargo( | 25 | pub fn load_cargo( |
26 | root: &Path, | 26 | root: &Path, |
27 | load_out_dirs_from_check: bool, | 27 | load_out_dirs_from_check: bool, |
28 | with_proc_macro: bool, | 28 | with_proc_macro: bool, |
diff --git a/editors/code/src/commands/ssr.ts b/editors/code/src/commands/ssr.ts index 6fee051fd..4ef8cdf04 100644 --- a/editors/code/src/commands/ssr.ts +++ b/editors/code/src/commands/ssr.ts | |||
@@ -11,7 +11,7 @@ export function ssr(ctx: Ctx): Cmd { | |||
11 | 11 | ||
12 | const options: vscode.InputBoxOptions = { | 12 | const options: vscode.InputBoxOptions = { |
13 | value: "() ==>> ()", | 13 | value: "() ==>> ()", |
14 | prompt: "EnteR request, for example 'Foo($a:expr) ==> Foo::new($a)' ", | 14 | prompt: "Enter request, for example 'Foo($a:expr) ==> Foo::new($a)' ", |
15 | validateInput: async (x: string) => { | 15 | validateInput: async (x: string) => { |
16 | try { | 16 | try { |
17 | await client.sendRequest(ra.ssr, { query: x, parseOnly: true }); | 17 | await client.sendRequest(ra.ssr, { query: x, parseOnly: true }); |