Skip to content

Adopt /swiftlang CI workflows - #536

Open
kukushechkin wants to merge 6 commits into
mainfrom
adopt-swiftlang-ci
Open

kukushechkin wants to merge 6 commits into
mainfrom
adopt-swiftlang-ci

Conversation

@kukushechkin

@kukushechkin kukushechkin commented Nov 13, 2026 •

Copy link
Copy Markdown
Contributor

Switch from swift-nio to swiftlang CI workflows. As there is no tag yet available, let's use the latest hash for now.

Motivation:

Modernize and unify workflows with swiftlang.

Modifications:

CI configuration updated.

Result:

No swift-nio references in CI workflows.
Additionally, Swift tools version triplet is now supported to unblock #534

@kukushechkin kukushechkin added the semver/none No version bump required. label Nov 13, 2026
@kukushechkin
kukushechkin marked this pull request as ready for review November 13, 2026 11:59
Comment thread .github/workflows/main.yml Outdated
enable_macos_swiftly: false
enable_freebsd: false

release-builds:

@rnro rnro Nov 13, 2026 •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

If you want you can roll this into the above block like:

      linux_command: |
        Debug-Test: swift test
        Release-Build: swift build -c release
      windows_command: |
        Debug-Test: swift test
        Release-Build: swift build -c release

They will be broken out into separate jobs because they are labeled (Foo: bar)

windows_command: |
Debug-Test:
command: swift test -Xswiftc -warnings-as-errors --explicit-target-dependency-import-check error -Xswiftc -require-explicit-sendable
versions: ["current-2", "current-1", "current"]

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

These should be selected by default, is that not working?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The old swift-nio based behavior is different commands (with/without -warnings-as-errors) and the default seems to include all 5.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Command diff should be more visible now with a shared var.

@kukushechkin
kukushechkin requested a review from rnro November 13, 2026 17:22
Debug-Test: swift test
Release-Build: swift build -c release
windows_env_vars: '{"COMMON_SWIFT_FLAGS": "--explicit-target-dependency-import-check error -Xswiftc -require-explicit-sendable"}'
windows_command: |

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think this should get you equivalent jobs created:

        linux_command: |
          Debug-Test: swift test
          Release-Build: swift build -c release

        windows_command: |
          Debug-Test: swift test
          Release-Build: swift build -c release

        windows_use_docker: true

        linux_version_overrides: |
          default: -Xswiftc -warnings-as-errors --explicit-target-dependency-import-check error -Xswiftc -require-explicit-sendable
          nightly-release: --explicit-target-dependency-import-check error -Xswiftc -require-explicit-sendable
          nightly-main: --explicit-target-dependency-import-check error -Xswiftc -require-explicit-sendable

        windows_version_overrides: |
          default: -Xswiftc -warnings-as-errors --explicit-target-dependency-import-check error -Xswiftc -require-explicit-sendable
          nightly-release: --explicit-target-dependency-import-check error -Xswiftc -require-explicit-sendable
          nightly-main: --explicit-target-dependency-import-check error -Xswiftc -require-explicit-sendable

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver/none No version bump required.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants