Skip to content

Conversation

@gjcairo
Copy link
Collaborator

@gjcairo gjcairo commented Jan 14, 2026

There are several formatting issues that need to be fixed:

  • YAML formatting
  • Missing/incorrect license headers
  • forEach formatting rule needs to be disabled in a couple of places

@gjcairo gjcairo requested a review from aryan-25 January 14, 2026 15:34
@gjcairo gjcairo added the semver/none No version bump required. label Jan 14, 2026
Copy link
Collaborator

@aryan-25 aryan-25 left a comment

Choose a reason for hiding this comment

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

Looks good. The format check in CI is failing due to swiftlang/swift-format#1081.

@gjcairo
Copy link
Collaborator Author

gjcairo commented Jan 19, 2026

The formatting check will fail because only the nightly formatter understands nonisolated(nonsending) and we're using 6.2. For now we'll have to live with it failing.

@gjcairo gjcairo requested a review from aryan-25 January 19, 2026 11:23
@gjcairo gjcairo merged commit 08dd8bf into main Jan 19, 2026
18 of 26 checks passed
@gjcairo gjcairo deleted the formatting branch January 19, 2026 11:24
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