Skip to content
This repository was archived by the owner on Nov 29, 2025. It is now read-only.

Conversation

@beeauvin
Copy link
Owner

Doesn't test caching but at least gets us closer to making sure the right version is installed :P

Doesn't test caching but at least gets us closer to making sure the right version is installed :P
Copilot AI review requested due to automatic review settings November 26, 2025 23:40
Copilot finished reviewing on behalf of beeauvin November 26, 2025 23:42
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR enhances the testing framework for the Swiftly Swift action by adding version verification capabilities. The action now exposes the resolved Swift version as an output, allowing consumers and tests to verify that the correct version was installed.

  • Adds a swift-version output to the action that exposes the resolved version
  • Updates tests to verify the installed Swift version matches the resolved version output
  • Expands test matrix to include major version "6" alongside "latest" and "6.1"

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
action.yml Adds swift-version output that references the existing resolve-version step
.github/workflows/tests.yml Updates test matrix to include version "6" and adds verification logic using the new output to confirm correct Swift version installation

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

@beeauvin beeauvin merged commit 08d1ca3 into main Nov 26, 2025
12 checks passed
@beeauvin beeauvin deleted the testing/validate-version-in-test branch November 26, 2025 23:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants