Skip to content

fix: move swift-tools-version to first line of Package.swift#479

Merged
DennisAlund merged 1 commit intooddbit:mainfrom
SalihCanBinboga:patch-1
Apr 7, 2026
Merged

fix: move swift-tools-version to first line of Package.swift#479
DennisAlund merged 1 commit intooddbit:mainfrom
SalihCanBinboga:patch-1

Conversation

@SalihCanBinboga
Copy link
Copy Markdown
Contributor

Swift 6.0+ requires the swift-tools-version declaration to be on the very first
line of Package.swift. The current file has a copyright header before it, which
causes Xcode (Swift 6.0+) to fail with:

the manifest is backward-incompatible with Swift < 6.0 because the tools-version
was specified in a subsequent line of the manifest, not the first line.

This moves the // swift-tools-version: 5.9 comment to line 1, before the copyright
header, following the Swift Package Manager specification.

Remove commented swift-tools-version line.
Copy link
Copy Markdown

@claude claude bot left a comment

Choose a reason for hiding this comment

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

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

@nstosic
Copy link
Copy Markdown

nstosic commented Apr 1, 2026

I'm running into the same issue, so I also want to raise urgency on getting this merged and published

Copy link
Copy Markdown
Contributor

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

Updates the Swift Package manifest header ordering to comply with Swift 6.0+ requirements that the swift-tools-version declaration appear on the very first line of Package.swift, preventing Xcode/SwiftPM manifest compatibility errors.

Changes:

  • Moved // swift-tools-version: 5.9 to line 1 in ios/facebook_app_events/Package.swift.
  • Kept the existing copyright/license header immediately after the tools version line.

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

@DennisAlund DennisAlund merged commit aaf64fc into oddbit:main Apr 7, 2026
5 checks passed
@DennisAlund
Copy link
Copy Markdown
Member

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.

4 participants