Skip to content

Bumped xcodeproj version to support PBXFileSystemSynchronizedGroupBuildPhaseMembershipExceptionSet#154

Open
JanC wants to merge 2 commits intobloomberg:mainfrom
JanC:fix-xcode-26
Open

Bumped xcodeproj version to support PBXFileSystemSynchronizedGroupBuildPhaseMembershipExceptionSet#154
JanC wants to merge 2 commits intobloomberg:mainfrom
JanC:fix-xcode-26

Conversation

@JanC
Copy link

@JanC JanC commented Jan 30, 2026

hey,

small PR to support the xcode projects with the PBXFileSystemSynchronizedGroupBuildPhaseMembershipExceptionSet

Describe your changes

Testing performed

Additional context

I the current master, I was getting the error

xcdiff  -p1 Proj1.xcodeproj -p2 Proj2.xcodeproj -v
ERROR: Encountered unknown error while loading XcodeProj at Proj1.xcodeproj

I debugged in the sources a found out it's becase of an unspupported xcode element.

I added a the error to the log message:

ERROR: Encountered unknown error while loading XcodeProj at Proj1.xcodeproj: The element PBXFileSystemSynchronizedGroupBuildPhaseMembershipExceptionSet is not supported.

@marciniwanicki
Copy link
Contributor

Thank you @JanC for the PR, very much appreciated! It looks like the DCO check is failing, would you mind signing your commits?

throw ComparatorError.generic(error.description)
} catch {
throw ComparatorError.generic("Encountered unknown error while loading XcodeProj at \(path.string)")
throw ComparatorError.generic("Encountered unknown error while loading XcodeProj at \(path.string): \(error)")
Copy link
Contributor

Choose a reason for hiding this comment

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

Perhaps we could change the ':' to a '.' here. Breaking this into two sentences makes the error output feel more like natural language and keeps it consistent with our other error message formats.

@marciniwanicki
Copy link
Contributor

@JanC, our CI requires all the commits to be signed. Please follow the DCO instruction https://github.com/bloomberg/xcdiff/pull/154/checks?check_run_id=62382920209. Thank you and sorry for all the troubles.

@JanC
Copy link
Author

JanC commented Feb 5, 2026

hey, yep I noticed, sorry for the delay. I'll fix it today

EDIT: fixed :)

@JanC JanC force-pushed the fix-xcode-26 branch 6 times, most recently from a6387ef to ade5e8f Compare February 5, 2026 07:05
JanC added 2 commits February 5, 2026 08:06
…ldPhaseMembershipExceptionSet

Signed-off-by: Jan Chaloupecky <jan.chaloupecky@gmail.com>
Signed-off-by: Jan Chaloupecky <jan.chaloupecky@gmail.com>
@codecov
Copy link

codecov bot commented Feb 6, 2026

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 96.84%. Comparing base (c48fb45) to head (c98dea7).
⚠️ Report is 15 commits behind head on main.

Files with missing lines Patch % Lines
Sources/XCDiffCore/Library/XcodeProjLoader.swift 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #154      +/-   ##
==========================================
- Coverage   97.05%   96.84%   -0.21%     
==========================================
  Files          52       57       +5     
  Lines        2986     2567     -419     
==========================================
- Hits         2898     2486     -412     
+ Misses         88       81       -7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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