Skip to content

Conversation

rnro
Copy link
Contributor

@rnro rnro commented Nov 11, 2024

Migrate CI to use GitHub Actions.

Motivation:

To migrate to GitHub actions and centralised infrastructure.

Modifications:

Changes of note:

  • Bump the minimum version to Swift 5.9 in line with CI coverage
  • Adopt swift-format using rules from SwiftNIO
  • Remove scripts which are no longer needed

Result:

GitHub Actions CI

Future improvements:

  • Re-enable --warnings-as-errors
  • Investigate extending re-used workflows to enable specifying setup commands and services to allow us to drop the bespoke workflows here.
  • Enable API breakage checks
  • Set thresholds and enable Benchmarks
  • Enable documentation checking

@rnro rnro added the 🆕 semver/minor Adds new public API. label Nov 11, 2024
@rnro rnro force-pushed the adopt_github_actions branch 8 times, most recently from a7f96fe to 6a8f918 Compare November 12, 2024 10:10
@FranzBusch
Copy link
Contributor

The license header check is failing

@rnro rnro force-pushed the adopt_github_actions branch from d9001f0 to b317d1d Compare November 12, 2024 13:18
rnro added 3 commits November 12, 2024 13:20
Migrate CI to use GitHub Actions.

Motivation:

To migrate to GitHub actions and centralised infrastructure.

Modifications:

Changes of note:
* Bump the minimum version to Swift 5.9 in line with CI coverage
* Adopt swift-format using rules from SwiftNIO
* Remove scripts which are no longer needed

Result:

Feature parity with old CI.

Future improvements:
* Enable API breakage checks
* Set thresholds and enable Benchmarks
@rnro rnro force-pushed the adopt_github_actions branch from dc393cc to aa24266 Compare November 13, 2024 09:19
@FranzBusch FranzBusch merged commit e768c1e into main Nov 13, 2024
23 checks passed
@FranzBusch FranzBusch deleted the adopt_github_actions branch November 13, 2024 09:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🆕 semver/minor Adds new public API.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants