Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate to GitHub Actions #151

Merged
merged 1 commit into from
Nov 6, 2024
Merged

Migrate to GitHub Actions #151

merged 1 commit into from
Nov 6, 2024

Conversation

rnro
Copy link
Contributor

@rnro rnro commented Oct 30, 2024

Migrate CI to use GitHub Actions.

Motivation:

To migrate to GitHub actions and centralised infrastructure.

Modifications:

Changes of note:

  • Adopt swift-format using rules from SwiftNIO
  • Remove scripts and docker files which are no longer needed

Result:

Feature parity with old CI.

@rnro rnro force-pushed the adopt_github_actions branch from fddf7d6 to 0411de5 Compare October 31, 2024 07:54
@rnro rnro added the 🔨 semver/patch No public API change. label Oct 31, 2024
Comment on lines 28 to 31
swift-6-language-mode:
name: Swift 6 Language Mode
uses: apple/swift-nio/.github/workflows/swift_6_language_mode.yml@main
if: false # Disabled for now.
Copy link
Member

Choose a reason for hiding this comment

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

I think we have been inconsistent with this but IIRC the latest we decided is to not add this to PRs at all. Maybe we should do this here as well.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, fine with me 👍

Migrate CI to use GitHub Actions.

Motivation:

To migrate to GitHub actions and centralised infrastructure.

Modifications:

Changes of note:
* Adopt swift-format using rules from SwiftNIO
* Remove scripts and docker files which are no longer needed

Result:

Feature parity with old CI.
@rnro rnro force-pushed the adopt_github_actions branch from 0411de5 to 03ffcb5 Compare November 1, 2024 10:14
@ktoso ktoso closed this Nov 6, 2024
@ktoso ktoso reopened this Nov 6, 2024
@ktoso
Copy link
Member

ktoso commented Nov 6, 2024

LGTM, i switched all the branch protection rules.

@ktoso ktoso merged commit d720898 into main Nov 6, 2024
52 checks passed
@ktoso ktoso deleted the adopt_github_actions branch November 6, 2024 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔨 semver/patch No public API change.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants