Skip to content

Add Ubuntu 26.04 and Debian 13 to the PR checks - #585

Merged
cmcgee1024 merged 14 commits into
swiftlang:mainfrom
cmcgee1024:add_pr_check_ubuntu26_debian13
Sep 26, 2026
Merged

cmcgee1024 merged 14 commits into
swiftlang:mainfrom
cmcgee1024:add_pr_check_ubuntu26_debian13

Conversation

@cmcgee1024

Copy link
Copy Markdown
Member

No description provided.

Comment thread .github/workflows/pull_request.yml Outdated
fail-fast: false
matrix:
container: ["swiftlang/swift:nightly-6.3-fedora41"]
container: ["swiftlang/swift:nightly-6.3-fedora41", "swiftlang/swift:nightly-6.3-resolute", "swiftlang/swift:nightly-6.3-trixie"]

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should switch over to official release of 6.4.0

@cmcgee1024
cmcgee1024 marked this pull request as ready for review September 24, 2026 23:08
Comment thread Sources/LinuxPlatform/Linux.swift Outdated
arguments: ["--version"],
output: .discarded
)
var gpgIsMissing = false

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

definite initialization should allow you to defer assigning to a let so long as you've assigned it before use:

            let gpgIsMissing: Bool

            do {
                let result = try await run(
                    .name("gpg"),
                    arguments: ["--version"],
                    output: .discarded
                )
                gpgIsMissing = !result.terminationStatus.isSuccess
            } catch let error as SubprocessError where error.code == .executableNotFound {
                gpgIsMissing = true
            }

Then we can make it immutable.

@cmcgee1024
cmcgee1024 merged commit c8cf2e3 into swiftlang:main Sep 26, 2026
27 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