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

GitHub Actions workflow fixes #422

Merged
merged 30 commits into from
Jul 10, 2024
Merged

Conversation

tonyarnold
Copy link
Collaborator

@tonyarnold tonyarnold commented Jul 9, 2024

Apologies for the notification spam while I was working on this issue.

This PR proposes fixes for some of the workflow issues introduced with #418, and to update the GitHub Actions workflows where appropriate to use recent versions of Xcode, macOS and the Actions plugins.

This ended up being a wonderful confluence of problems to solve, resulting in the following changes:

  • macOS 11 runners have been removed (GHA doesn't support them anymore).
  • Swift versions prior to 5.7 on platforms other than macOS had to be removed, as the images were conflicting with the actions/checkout action (in reality, all node-based actions would fail on these older images).
  • Where possible, base tasks like SwiftLint and CocoaPods verification happen on a macOS 14 Sonoma image running Xcode 15.4 (the latest at the time of authoring).
  • CocoaPods needed to be upgraded for visionOS support.
  • A few minor SwiftLint violations introduced in recent versions needed to be fixed or disabled.
  • I disabled running SwiftLint from within the Xcode project when it detects that it is being built on CI. It seemed unnecessary given it's running in a dedicated workflow.
  • Existing workflow jobs will now auto-cancel if a new set of jobs is started on the same branch.

@tonyarnold tonyarnold marked this pull request as ready for review July 9, 2024 02:20
@tonyarnold
Copy link
Collaborator Author

😅 this is going to take some elbow grease.

@tonyarnold tonyarnold force-pushed the visionos-workflow-fixes branch 4 times, most recently from ee30155 to cff2a37 Compare July 10, 2024 02:50
@tonyarnold tonyarnold changed the title visionOS workflow fixes GitHub Actions workflow fixes Jul 10, 2024
@tonyarnold tonyarnold force-pushed the visionos-workflow-fixes branch from cd15f44 to 380c015 Compare July 10, 2024 03:26
@tonyarnold tonyarnold force-pushed the visionos-workflow-fixes branch from 0b5e666 to 91044c1 Compare July 10, 2024 03:36
@tonyarnold tonyarnold force-pushed the visionos-workflow-fixes branch from ff5657f to c31c96b Compare July 10, 2024 04:21
@tonyarnold tonyarnold force-pushed the visionos-workflow-fixes branch from c31c96b to 2ecc035 Compare July 10, 2024 04:22
@tonyarnold tonyarnold requested a review from jpsim July 10, 2024 04:59
@tonyarnold
Copy link
Collaborator Author

Okay, I believe everything is now running on CI and passing out of the box. I'm more than happy to consolidate commits, or make further changes — just let me know!

jpsim added 4 commits July 10, 2024 08:43
We should only have one of these running and it's already configured
for the latest Xcode version on macOS Sonoma.
Since 5.1.3 isn't released yet.
@jpsim
Copy link
Owner

jpsim commented Jul 10, 2024

Amazing work here @tonyarnold, just a heroic amount of effort to slog through these CI updates. I can't thank you enough for these contributions.

I pushed some small changes to your branch and will merge + cut a 5.1.3 release once that passes CI (🤞🤞🤞).

@jpsim jpsim merged commit 98650c6 into jpsim:main Jul 10, 2024
45 checks passed
@tonyarnold tonyarnold deleted the visionos-workflow-fixes branch July 10, 2024 13:17
jpsim added a commit to finagolfin/Yams that referenced this pull request Jul 10, 2024
* main:
  GitHub Actions workflow fixes (jpsim#422)
  add os(visionOS) support (jpsim#418)
  Bump rexml from 3.2.5 to 3.2.8
jpsim added a commit to futurejones/Yams that referenced this pull request Jul 10, 2024
* main:
  Add support for Android (jpsim#421)
  GitHub Actions workflow fixes (jpsim#422)
  add os(visionOS) support (jpsim#418)
  Bazel: support rules_swift 2.x (jpsim#420)
  Bump rexml from 3.2.5 to 3.2.8
tejassharma96 added a commit to tejassharma96/Yams that referenced this pull request Jul 10, 2024
# By JP Simard (3) and others
# Via GitHub
* upstream/main:
  Require Swift 5.7 or later (jpsim#424)
  Add empty changelog section
  Release 5.1.3
  add support for riscv64 (jpsim#419)
  Add support for Android (jpsim#421)
  GitHub Actions workflow fixes (jpsim#422)
  add os(visionOS) support (jpsim#418)
  Bazel: support rules_swift 2.x (jpsim#420)
  Bump rexml from 3.2.5 to 3.2.8

# Conflicts:
#	CHANGELOG.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants