Skip to content

chore: rename salsa to ra_salsa#18239

Merged
bors merged 1 commit into
rust-lang:masterfrom
davidbarsky:davidbarsky/push-lswkwuyrpuqv
Oct 14, 2024
Merged

chore: rename salsa to ra_salsa#18239
bors merged 1 commit into
rust-lang:masterfrom
davidbarsky:davidbarsky/push-lswkwuyrpuqv

Conversation

@davidbarsky

@davidbarsky davidbarsky commented Oct 4, 2024

Copy link
Copy Markdown
Contributor

Laying some groundwork to start before I import the new Salsa crate. Here's why:

  1. As part of the migration, @darichey, @Wilfred, and I will create new Salsa equivalents of the existing databases/query groups. We'll get them to compile crate-by-crate.
  2. Once we wrote all equivalents of all queries, we'd start to refactor usage sites of the vendored Salsa to use the new Salsa databases.
  3. Starting porting usage sites of old Salsa to the new Salsa.
  4. Remove the vendored ra_salsa; declare victory.

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 4, 2024
Comment thread crates/ra-salsa/Cargo.toml Outdated
@davidbarsky davidbarsky force-pushed the davidbarsky/push-lswkwuyrpuqv branch from d6e4e9a to ccee36e Compare October 14, 2024 14:11
@davidbarsky

Copy link
Copy Markdown
Contributor Author

@bors r+

@bors

bors commented Oct 14, 2024

Copy link
Copy Markdown
Contributor

📌 Commit ccee36e has been approved by davidbarsky

It is now in the queue for this repository.

@bors

bors commented Oct 14, 2024

Copy link
Copy Markdown
Contributor

⌛ Testing commit ccee36e with merge 77d4376...

@bors

bors commented Oct 14, 2024

Copy link
Copy Markdown
Contributor

☀️ Test successful - checks-actions
Approved by: davidbarsky
Pushing 77d4376 to master...

@bors bors merged commit 77d4376 into rust-lang:master Oct 14, 2024
@davidbarsky davidbarsky deleted the davidbarsky/push-lswkwuyrpuqv branch October 14, 2024 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants