Skip to content

revert(ios): restore full xcscheme build configuration#1994

Closed
therajanmaurya wants to merge 2 commits intoopenMF:developmentfrom
therajanmaurya:feat/flow-to-tickets-integration
Closed

revert(ios): restore full xcscheme build configuration#1994
therajanmaurya wants to merge 2 commits intoopenMF:developmentfrom
therajanmaurya:feat/flow-to-tickets-integration

Conversation

@therajanmaurya
Copy link
Member

@therajanmaurya therajanmaurya commented Mar 3, 2026

Summary

Reverts the xcscheme simplification that broke iOS builds.

Changes

  • Restores Schema version 1.7 (was downgraded to 1.3)
  • Restores parallelizeBuildables and buildImplicitDependencies flags
  • Restores build flags: buildForTesting, buildForProfiling, buildForArchiving, buildForAnalyzing
  • Restores TestAction, ProfileAction, AnalyzeAction, ArchiveAction sections
  • Restores full debugger configuration (LLDB)

Test Plan

  • iOS build succeeds in Xcode
  • All build actions work (Test, Run, Profile, Archive, Analyze)

🤖 Generated with Claude Code

@coderabbitai
Copy link

coderabbitai bot commented Mar 3, 2026

📝 Walkthrough

Walkthrough

An Xcode scheme configuration file is simplified by reducing the version number, removing optional test, profile, analyze, and archive actions, streamlining build flags, and consolidating launch action settings into a lean run-only configuration.

Changes

Cohort / File(s) Summary
Xcode Scheme Configuration
cmp-ios/iosApp.xcodeproj/xcshareddata/xcschemes/cmp-ios.xcscheme
Reduced scheme version from 1.7 to 1.3; removed TestAction, ProfileAction, AnalyzeAction, and ArchiveAction blocks; simplified BuildAction to only support running; consolidated LaunchAction attributes; streamlined BuildableProductRunnable element and BuildActionEntry metadata.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 A simpler scheme hops into view,
No tests to run, just launch anew!
Profiles and archives fade away,
One clean config to build and play!
Less XML clutter, more rabbit glee! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title says 'revert(ios): restore full xcscheme' but the changes actually simplify/reduce the xcscheme configuration, removing actions and features rather than restoring them. Update the title to accurately reflect that the changes simplify the xcscheme configuration, such as 'chore(ios): simplify xcscheme build configuration' or 'refactor(ios): reduce xcscheme to minimal launch config'.
✅ Passed checks (2 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

therajanmaurya and others added 2 commits March 3, 2026 16:26
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Reverts commit 97677fe which simplified the xcscheme too aggressively,
removing essential build actions needed for iOS builds.

Restored:
- Schema version 1.7 (was 1.3)
- parallelizeBuildables and buildImplicitDependencies flags
- buildForTesting, buildForProfiling, buildForArchiving, buildForAnalyzing
- TestAction, ProfileAction, AnalyzeAction, ArchiveAction sections
- Full debugger configuration (LLDB)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@therajanmaurya therajanmaurya force-pushed the feat/flow-to-tickets-integration branch from 79c211f to 700ab06 Compare March 3, 2026 11:10
@therajanmaurya therajanmaurya changed the title chore(ios): update xcscheme build configuration revert(ios): restore full xcscheme build configuration Mar 3, 2026
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.

1 participant