Skip to content

add swift 5.8 & 5.9 workflow#125

Merged
NeedleInAJayStack merged 1 commit into
GraphQLSwift:mainfrom
NoDevOrg:swift-5.9
Nov 6, 2023
Merged

add swift 5.8 & 5.9 workflow#125
NeedleInAJayStack merged 1 commit into
GraphQLSwift:mainfrom
NoDevOrg:swift-5.9

Conversation

@samisuteria

@samisuteria samisuteria commented Oct 12, 2023

Copy link
Copy Markdown
Contributor

Also ran swiftformat since the workflow was failing.

@NeedleInAJayStack NeedleInAJayStack left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Ugh, thanks for running the formatter. It's frustrating that it doesn't seem to stay consistent across versions.

Thanks!

@NeedleInAJayStack

Copy link
Copy Markdown
Member

@samisuteria Hey, I made some CI changes to fix the MacOS runner. Would you mind rebasing on main and add in the Swift 5.8 again? Thanks again for these changes.

@samisuteria

Copy link
Copy Markdown
Contributor Author

@NeedleInAJayStack made the changes :)

@NeedleInAJayStack NeedleInAJayStack self-requested a review November 6, 2023 21:17

@NeedleInAJayStack NeedleInAJayStack left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@NeedleInAJayStack NeedleInAJayStack merged commit 742dba5 into GraphQLSwift:main Nov 6, 2023
@samisuteria samisuteria deleted the swift-5.9 branch August 5, 2024 04:56
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