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

[CI] Adopt GitHub actions #280

Merged
merged 5 commits into from
Oct 16, 2024
Merged

[CI] Adopt GitHub actions #280

merged 5 commits into from
Oct 16, 2024

Conversation

FranzBusch
Copy link
Member

Motivation

We want to migrate all of repositories to use GitHub Actions.

Modification

This PR adopts GH actions in this repo.

Result

New and shiny CI.

@FranzBusch FranzBusch added the semver/none No version bump required. label Oct 16, 2024
@FranzBusch FranzBusch force-pushed the fb-gha branch 2 times, most recently from 68aa605 to 3d3461e Compare October 16, 2024 13:47
# Motivation

We want to migrate all of repositories to use GitHub Actions.

# Modification

This PR adopts GH actions in this repo.

# Result

New and shiny CI.
@FranzBusch
Copy link
Member Author

@Lukasa @simonjbeaumont this PR is ready. I have split this into multiple commits to make it easier to review

Copy link
Contributor

Choose a reason for hiding this comment

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

Why was this renamed and do we need to worry about anyone having linked to this file?

Copy link
Member Author

Choose a reason for hiding this comment

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

This was renamed to align it with the majority of our other repos. There was a weird inconsistency that I wasn't aware of before. I am not worried that anybody linked to it.

@FranzBusch FranzBusch merged commit ae321d2 into apple:main Oct 16, 2024
25 of 34 checks passed
@FranzBusch FranzBusch deleted the fb-gha branch October 16, 2024 17:19
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.

3 participants