Skip to content

Don't set the minimum Swift version floor at the patch level - #350

Merged
FranzBusch merged 1 commit into
swiftlang:mainfrom
rnro:patch-tools-version-minimum
Nov 10, 2026
Merged

FranzBusch merged 1 commit into
swiftlang:mainfrom
rnro:patch-tools-version-minimum

Conversation

@rnro

@rnro rnro commented Nov 10, 2026

Copy link
Copy Markdown
Contributor

A manifest with swift-tools-version: 6.2.3 produced a floor of 6.2.3, while the 6.2 version label orders as 6.2.0. The filter then skipped Swift 6.2 even though the 6.2 toolchain satisfies that tools version.

Changed:

  • Truncated the floor in MinimumVersion to its major and minor components.
  • Added tests for a patch tools version and a patch minimum_swift_version.

Result:

  • A patch-level minimum keeps its own minor release in the matrix.

A manifest with `swift-tools-version: 6.2.3` produced a floor of `6.2.3`, while the `6.2` version label orders as `6.2.0`. The filter then skipped Swift 6.2 even though the `6.2` toolchain satisfies that tools version.

Changed:
* Truncated the floor in `MinimumVersion` to its major and minor components.
* Added tests for a patch tools version and a patch `minimum_swift_version`.

Result:
* A patch-level minimum keeps its own minor release in the matrix.
@rnro
rnro requested a review from a team as a code owner November 10, 2026 13:53
@FranzBusch
FranzBusch merged commit de9be12 into swiftlang:main Nov 10, 2026
221 of 224 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants