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: Migrate to GitHub Actions and reusable workflows, part two #117

Merged

Conversation

simonjbeaumont
Copy link
Collaborator

@simonjbeaumont simonjbeaumont commented Oct 2, 2024

Motivation

Following #116 we can now cut over to the soundness checks from the GithHub Actions CI and short-circuit the old CI checks.

Modifications

  • Remove most scripts used by old CI
  • Short-circuit API checking script
  • Short-circuit the docker CI
  • Enable soundness tests in GitHub Actions workflow
  • Ignore docker/* for license check
  • Remove DocC plugin from package manifest
  • Update CONTRIBUTING.md with instructions for local run

Result

GitHub Actions CI is the one that we care about. We can then update the branch rules, disable the old webhook, and, finally, remove the vestigial stuff.

@simonjbeaumont simonjbeaumont changed the title Sb/GitHub actions migration 2 ci: Migrate to GitHub Actions and reusable workflows, part two Oct 2, 2024
@simonjbeaumont simonjbeaumont marked this pull request as ready for review October 2, 2024 11:06
@simonjbeaumont simonjbeaumont merged commit f795237 into apple:main Oct 2, 2024
25 of 26 checks passed
simonjbeaumont added a commit that referenced this pull request Oct 2, 2024
…al) (#118)

### Motivation

Following on from #116 and #117, we have now disabled the webhook that
was driving the old CI and updated the branch protection rules to
require the new GitHub Actions based CI checks.

We can now delete the final shims that were keeping the old CI green,
since it is no longer running and complete the migration.

### Modifications

- Delete all Docker bits and scripts used by old CI.

### Result

Migration complete.

### Test Plan

We should be seeing _only_ GitHub Actions checks on this PR and they
should all be passing (apart from the ones that are explicitly
disabled).
@czechboy0 czechboy0 added the semver/none No version bump required. label Oct 18, 2024
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