Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

improvement: bump sqlx to 0.8 in database examples #181

Merged
merged 4 commits into from
Aug 15, 2024
Merged

improvement: bump sqlx to 0.8 in database examples #181

merged 4 commits into from
Aug 15, 2024

Conversation

sentinel1909
Copy link
Contributor

Description of change

  • this is a matching PR to #1838 submitted for the main Shuttle repo
  • bumps sqlx to 0.8 in all database examples

How has this been tested? (if applicable)

  • have not done any testing
  • I believe that the PR for #1838 in the main repo must be accepted first

@jonaro00 jonaro00 changed the base branch from main to develop July 29, 2024 09:50
sqlx = { version = "0.7.3", features = ["runtime-tokio-native-tls", "postgres"] }
shuttle-runtime = "0.46.0"
shuttle-shared-db = { version = "0.46.0", features = ["postgres"] }
sqlx = { version = "0.7.1", features = ["runtime-tokio-native-tls", "postgres"] }
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This has to stay on a lower version of runtime+shared-db for a while since the cargo+sqlx feature bug pulls in 2 versions of libsqlite. Seems like it's impossible to support both versions (like for axum) due to it. cc @oddgrd

@sentinel1909
Copy link
Contributor Author

sentinel1909 commented Jul 30, 2024

Got it. Do you want me to make the change and submit a new PR? It looks like it's already taken care of.

@jonaro00
Copy link
Member

It should be all done. Just sharing context.

@jonaro00 jonaro00 merged commit db77f14 into shuttle-hq:develop Aug 15, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants