Skip to content

Support new Swift Xcode Project type#1491

Merged
a-maurice merged 9 commits into
mainfrom
am-xcode_swift_project
Jul 15, 2026
Merged

Support new Swift Xcode Project type#1491
a-maurice merged 9 commits into
mainfrom
am-xcode_swift_project

Conversation

@a-maurice

Copy link
Copy Markdown
Collaborator

Description

Provide details of the change, and generalize the change in the PR title above.

Unity 6.5+ has a new Swift Project type setting, that among other things changes how the generated Xcode project is named. To handle this, use the method PBXProject.GetPBXProjectPath instead.

More information on the setting can be found here: https://docs.unity3d.com/6000.5/Documentation/Manual/ios-introducing-swift.html


Testing

Describe how you've tested these changes.

Testing manually, along with the necessary EDM4U change that was just released.


Type of Change

Place an x the applicable box:

  • Bug fix. Add the issue # below if applicable.
  • New feature. A non-breaking change which adds functionality.
  • Other, such as a build process or documentation change.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request updates the iOS post-build process for Crashlytics to dynamically resolve the PBX project path using PBXProject.GetPBXProjectPath instead of a hardcoded path. The review feedback correctly identifies a typo in the newly added comment, which refers to GetPBXProjectPath as a 'Private class' instead of a private method.

Comment thread editor/crashlytics/src/iOSPostBuild.cs Outdated
@a-maurice a-maurice requested a review from AustinBenoit July 14, 2026 21:24
@a-maurice a-maurice merged commit f94d739 into main Jul 15, 2026
13 checks passed
@a-maurice a-maurice deleted the am-xcode_swift_project branch July 15, 2026 20:30
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