Skip to content

drop pre-sqlx schemamama compat migration layer - #3472

Merged
GuillaumeGomez merged 1 commit into
rust-lang:mainfrom
syphar:pre-sqlx
Aug 26, 2026
Merged

drop pre-sqlx schemamama compat migration layer#3472
GuillaumeGomez merged 1 commit into
rust-lang:mainfrom
syphar:pre-sqlx

Conversation

@syphar

@syphar syphar commented Aug 26, 2026

Copy link
Copy Markdown
Member
  • we only care about our deployment, no self-deployment
  • the migration sqlx is already a couple of years in the past.

@syphar syphar self-assigned this Aug 26, 2026
@syphar
syphar requested a review from a team as a code owner August 26, 2026 09:23
@github-actions github-actions Bot added the S-waiting-on-review Status: This pull request has been implemented and needs to be reviewed label Aug 26, 2026
@syphar

syphar commented Aug 26, 2026

Copy link
Copy Markdown
Member Author

r? @GuillaumeGomez

@rustbot rustbot assigned GuillaumeGomez and unassigned syphar Aug 26, 2026
@GuillaumeGomez

Copy link
Copy Markdown
Member

CI failed.

@rustbot

rustbot commented Aug 26, 2026

Copy link
Copy Markdown
Collaborator

This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

@syphar

syphar commented Aug 26, 2026

Copy link
Copy Markdown
Member Author

CI failed.

didn't wait long enough :) fixed. @GuillaumeGomez

@GuillaumeGomez

Copy link
Copy Markdown
Member

Code removal! \o/

@GuillaumeGomez
GuillaumeGomez merged commit e959175 into rust-lang:main Aug 26, 2026
15 checks passed
@GuillaumeGomez
GuillaumeGomez deleted the pre-sqlx branch August 26, 2026 10:43
@github-actions github-actions Bot added S-waiting-on-deploy This PR is ready to be merged, but is waiting for an admin to have time to deploy it and removed S-waiting-on-review Status: This pull request has been implemented and needs to be reviewed labels Aug 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-deploy This PR is ready to be merged, but is waiting for an admin to have time to deploy it

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants