Skip to content

Commit b00cb99

Browse files
committed
fix: matches_prerelease should be only used on OptVersionReq::Req
1 parent 2c27615 commit b00cb99

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

src/cargo/util/semver_ext.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,7 @@ impl OptVersionReq {
116116
/// The logic here is temporary, we'll have to consider more boundary conditions later,
117117
/// and we're not sure if this part of the functionality should be implemented in semver or cargo.
118118
pub fn matches_prerelease(&self, version: &Version) -> bool {
119-
if version.is_prerelease() {
119+
if version.is_prerelease() && matches!(self, OptVersionReq::Req(_)) {
120120
let mut version = version.clone();
121121
version.pre = semver::Prerelease::EMPTY;
122122
return self.matches(&version);
@@ -251,9 +251,9 @@ mod matches_prerelease {
251251
assert!(req.matches_prerelease(&to_ver));
252252

253253
let req = OptVersionReq::Locked(to_ver.clone(), req_ver.clone());
254-
assert!(!req.matches_prerelease(&to_ver));
254+
assert!(req.matches_prerelease(&to_ver));
255255

256256
let req = OptVersionReq::Locked(to_ver.clone(), req_ver.clone());
257-
assert!(!req.matches_prerelease(&to_ver));
257+
assert!(req.matches_prerelease(&to_ver));
258258
}
259259
}

0 commit comments

Comments
 (0)