[CI] Drop xcframeworks artifacts - #1213
Conversation
📝 WalkthroughWalkthroughRelease automation no longer updates framework artifact mappings or uploads iOS archive assets. The artifact mapping file is cleared, and the upcoming changelog documents the removal of CocoaPods and XCFrameworks support. ChangesRelease artifact cleanup
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
fastlane/Fastfile (1)
85-90: 🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick winDisable CocoaPods publishing explicitly.
publish_ios_sdkstill defaults to CocoaPods publishing (skip_pods: false), while archive/podspec publishing is being removed. Addskip_pods: true, or keep CocoaPods handling separate if it is intentionally retained.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@fastlane/Fastfile` around lines 85 - 90, Update the publish_ios_sdk invocation to explicitly set skip_pods: true, disabling CocoaPods publishing while preserving the existing release version, SDK names, and repository arguments.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Outside diff comments:
In `@fastlane/Fastfile`:
- Around line 85-90: Update the publish_ios_sdk invocation to explicitly set
skip_pods: true, disabling CocoaPods publishing while preserving the existing
release version, SDK names, and repository arguments.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: 08b03292-e959-42ba-9ccc-d15a06504d90
📒 Files selected for processing (3)
CHANGELOG.mdStreamVideoArtifacts.jsonfastlane/Fastfile
💤 Files with no reviewable changes (1)
- StreamVideoArtifacts.json
# Conflicts: # CHANGELOG.md # StreamVideoArtifacts.json
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
CHANGELOG.md (1)
7-7: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick winFix the heading hierarchy.
### 🔄 Changedskips directly from the parent# Upcomingheading to level three. Use## 🔄 Changedhere, or otherwise make the surrounding changelog hierarchy consistent.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@CHANGELOG.md` at line 7, Update the changelog heading hierarchy by changing the “🔄 Changed” heading under “# Upcoming” from level three to level two, keeping the surrounding section structure consistent.Source: Linters/SAST tools
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Outside diff comments:
In `@CHANGELOG.md`:
- Line 7: Update the changelog heading hierarchy by changing the “🔄 Changed”
heading under “# Upcoming” from level three to level two, keeping the
surrounding section structure consistent.
Summary by CodeRabbit