-
-
Notifications
You must be signed in to change notification settings - Fork 553
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
PostgreSQL Ipnetwork support #2395
base: master
Are you sure you want to change the base?
Conversation
fad19c8
to
0ee6c47
Compare
0ee6c47
to
e099f56
Compare
e099f56
to
fee320f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thank you. I think it looks good
Recently, Does anyone have any thought on how to deal with this issue? |
fee320f
to
ebd700a
Compare
I resolved the conflicts and rebased on the latest commit.
I think this required to wait for the upstream release: launchbadge/sqlx#3670. Then, wait for sea-query to publish a release. |
The way Diesel does this is by allowing more than one version: https://github.com/diesel-rs/diesel/blob/a5325d0d70c33c1952448121e1e52f4f5be6e9e5/diesel/Cargo.toml#L36 ipnetwork = { version = ">=0.12.2, <0.22.0", optional = true } |
TODO:
std::net::IpAddr
support?ipnet::IpNet
support?sea-ql
to support firstPR Info
inet
) #888New Features
with-ipnetwork