Skip to content

Conversation

rnro
Copy link
Contributor

@rnro rnro commented Nov 12, 2024

Migrate CI to use GitHub Actions.

Motivation:

To migrate to GitHub actions and centralised infrastructure.

Modifications:

Changes of note:

  • Bump minimum Swift version to 5.9 in line with CI coverage.
  • Update license header wording.
  • Add CONTRIBUTING.md.
  • Remove scripts and docker files which are no longer needed.

Result:

Feature parity with old CI plus additional soundness checks.

Migrate CI to use GitHub Actions.

Motivation:

To migrate to GitHub actions and centralised infrastructure.

Modifications:

Changes of note:
* Bump minimum Swift version to 5.9 in line with CI coverage.
* Update license header wording.
* Add CONTRIBUTING.md.
* Remove scripts and docker files which are no longer needed.

Result:

Feature parity with old CI plus additional soundness checks.
@rnro rnro added the 🆕 semver/minor Adds new public API. label Nov 12, 2024
Copy link
Collaborator

@ktoso ktoso left a comment

Choose a reason for hiding this comment

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

LGTM thank you

@FranzBusch FranzBusch merged commit 046eada into main Nov 12, 2024
26 of 27 checks passed
@FranzBusch FranzBusch deleted the adopt_github_actions branch November 12, 2024 11:51
@rnro rnro restored the adopt_github_actions branch November 12, 2024 13:21
@rnro rnro deleted the adopt_github_actions branch November 12, 2024 13:47
rnro added a commit that referenced this pull request Mar 31, 2025
Motivation:

To correct the license header changes made as part of #117.

Modifications:

* Revert license header changes
* Disable license header checks

Result:

Correct license headers.
@rnro rnro mentioned this pull request Mar 31, 2025
rnro added a commit that referenced this pull request Mar 31, 2025
### Motivation:

To correct the license header changes made as part of #117.

### Modifications:

* Revert license header changes
* License header checks use custom `.license_header_template`

### Result:

Correct license headers.
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.

3 participants