Skip to content

Conversation

@aumetra
Copy link
Contributor

@aumetra aumetra commented May 29, 2025

Analogue to #707 but for the postgres feature in Diesel.
Makes stuff easier when exclusively using diesel_async.

@Tony-Samuels Tony-Samuels merged commit 599dff9 into paupino:master May 29, 2025
15 checks passed
@Logarithmus
Copy link

Logarithmus commented Sep 24, 2025

@Tony-Samuels @paupino this commit somehow doesn't exist in 1.38.0 version of rust_decimal from crates.io.
rust_decimal still pulls pq-sys and it expects libpq-dev to be installed. I use diesel-async in my project with native Rust Postgres driver, so it pollutes my dependency tree.

image

https://docs.rs/crate/rust_decimal/1.38.0/features

@Tony-Samuels
Copy link
Collaborator

@Logarithmus , this is expected as the master branch is currently preparing for a (distant) v2. Changes aimed at the current v1 release need to be merged into the v1 branch.

Apologies for not pointing that out at the time I merged this, I've tried to do so consistently but evidently forgot to warn you about it this time.

@Tony-Samuels
Copy link
Collaborator

If you raise a similar MR against v1 I'll happily approve it and it will be made available in the next release.

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.

3 participants