Implementation for range with multiple look buged when pre-release are not present

[Stargateur]

These tests look wrong

semver/tests/test_version_req.rs

Lines 111 to 115 in 0080ed9

	let ref r = req(">1.0.0-alpha, <1.0");
	assert_match_none(r, &["1.0.0-beta"]);
	let ref r = req(">1.0.0-alpha, <1");
	assert_match_none(r, &["1.0.0-beta"]);

According to the comment:

semver/src/eval.rs

Lines 14 to 16 in 0080ed9

	// If a version has a prerelease tag (for example, 1.2.3-alpha.3) then it
	// will only be allowed to satisfy req if at least one comparator with the
	// same major.minor.patch also has a prerelease tag.

That should match, It's very strange no ? Specially:,

semver/tests/test_version_req.rs

Lines 150 to 161 in 0080ed9

	let ref r = req(">=0.5.1-alpha3, <0.6");
	assert_to_string(r, ">=0.5.1-alpha3, <0.6");
	assert_match_all(
	r,
	&[
	"0.5.1-alpha3",
	"0.5.1-alpha4",
	"0.5.1-beta",
	"0.5.1",
	"0.5.5",
	],
	);

Is as expected. And:

semver/tests/test_version_req.rs

Lines 108 to 109 in 0080ed9

	let ref r = req(">1.0.0-alpha, <1.0.0");
	assert_match_all(r, &["1.0.0-beta"]);

Clearly show that 1.0.0-alpha < 1.0.0-beta < 1.0.0) Do I miss something ?

[dtolnay]

This matches the proto-spec in semver/semver#584 and matches npm's semver implementation. The spec's treatment of <I.J is consistent with the behavior that I would want it to have.