You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I'm currently writing the v0.2 -> v0.3 upgrade guide and realizing we still have the name Range for our default impl of VersionSet. However, this is not exactly the same as the previous v0.2 Range.
So to prevent user confusion, and also make the upgrade guide clearer, I'd suggest we rename it to BoundedRange.
That's also more precise since it's a range built with inclusive/exclusive bounds.
The text was updated successfully, but these errors were encountered:
Hi @Michaelin007 thanks for your interest in this! The upgrade guide currently lives in the pubgrub-rs/guide repo, in a pull request I did not finish. pubgrub-rs/guide#6
You might have to coordinate with @Eh2406 and @charliermarsh who started reading that guide and leaving some comments.
I'm not currently working on the guide so no need to coordinate with me. I'm happy to answer questions though.
I'm currently writing the v0.2 -> v0.3 upgrade guide and realizing we still have the name
Range
for our default impl ofVersionSet
. However, this is not exactly the same as the previous v0.2Range
.So to prevent user confusion, and also make the upgrade guide clearer, I'd suggest we rename it to
BoundedRange
.That's also more precise since it's a range built with inclusive/exclusive bounds.
The text was updated successfully, but these errors were encountered: