Hugh sadly it seems like it changes the location of the problem, now sqlx complains because the version is wrong.
I am not sure what we should do in this, create features with different version ipnetwork19, ipnetwork20?
Since sqlx is the primary target for sea-query I guess we can also accept that the diesel integration will have a bad experience. @billy1624 @tyt2y3 if you have other ideas...

The cargo WG seems to go in the direction of recommending libraries to commit a lock file for the CI to work (rust-lang/cargo#8728), this is likely the way to go for the diesel integration (along with some documentation on why the user build might fail and the relevant cargo update --precise command) if we want to keep a fixed version of the dependencies. If this is acceptable I will close the PR and commit the lock file in #658.

We've not decided yet on Cargo.lock

However, we have added recommendations for version requirements

@epage Yeah I know, but until cargo introduces a way to mark public dependencies I don't see any other way to make my changes work :/
When you have

Lib -> Dep A -> Subdep
    -> Dep B -> Subdep

There is simply no way to ensure both Subdep will the same version for CI to work.

I understand, I was just wanting to clarify where we are currently at. I'm hopeful we'll be changing the guidelines soon. I know for my own projects, I have switched to committing my Cargo.lock files and I've talked to other maintainers who have done the same.

Also, we have been discussing scaling back the public-dependencies RFC to not affect package resolution, see rust-lang/rust#44663 (comment)