EMT-2244 Source Code Restructure and Cleanup #1504
Conversation
This reverts commit af22cad.
|
Summary By MatterAI
🔄 What ChangedThis Pull Request is a significant source code restructure and cleanup, primarily focused on the complete removal of the
All changes are deletions, indicating a deprecation or migration away from these specific tools and their associated build processes. 🔍 Impact of the ChangeThis PR represents a major refactoring of the project's build and CI/CD infrastructure. The removal of 📁 Total Files Changed8 files were changed, all of which were deletions. 🧪 Test AddedNo new tests were added or modified in this Pull Request. The changes exclusively involve the removal of existing build and CI/CD configuration files and helper scripts. 🔒 Security VulnerabilitiesNo new security vulnerabilities were introduced or fixed in the provided patch, as the changes consist solely of file removals. The implications for security would depend on the new build/CI system that replaces the removed Tip Quality Recommendations
Sequence DiagramsequenceDiagram
participant OldBuildSystem as "Old Build System"
participant FastlaneConfig as "Fastlane Configuration"
participant CocoapodsHelpers as "CocoaPods Helpers"
participant Gemfile as "Gemfile Dependencies"
participant Repository as "Code Repository"
Repository->>FastlaneConfig: Contains Fastlane Files
Repository->>CocoapodsHelpers: Contains Helper Scripts
Repository->>Gemfile: Defines Ruby Dependencies
OldBuildSystem->>FastlaneConfig: Relies on Fastlane for CI/CD
OldBuildSystem->>CocoapodsHelpers: Uses Helpers for Pod Management
OldBuildSystem->>Gemfile: Installs Ruby Gems
Note over Repository,OldBuildSystem: Source Code Restructure Cleanup (Removal Action)
Repository--xFastlaneConfig: Files Removed (Appfile, Fastfile, Matchfile, Scanfile)
Repository--xCocoapodsHelpers: Scripts Removed (cocoapods_helper.rb, version_helper.rb)
Repository--xGemfile: Dependencies Removed (Gemfile)
OldBuildSystem--xOldBuildSystem: Fastlane/CocoaPods Integration Decommissioned
|
|
Important PR Review SkippedPR review skipped as per the configuration setting. Run a manually review by commenting /matter review 💡Tips to use Matter AICommand List
|
|
Note PR Review SkippedPR review skipped as no relevant changes found due to large diff hunk OR part of a non-reviewable file. 📄Files skipped in review
💡Tips to use Matter AICommand List
|
|
Important PR Review SkippedPR review skipped as per the configuration setting. Run a manually review by commenting /matter review 💡Tips to use Matter AICommand List
|
NidhiDixit09
changed the title
NidhiDixit09
changed the title
…de-Restructure-Cleanup
|
Note PR Review SkippedPR review skipped as no relevant changes found due to large diff hunk OR part of a non-reviewable file. 📄Files skipped in review
💡Tips to use Matter AICommand List
|
Reference
EMT-2244 - Remove extra test Apps from repo and moved unit tests to BranchSDK project file.
https://branch.atlassian.net/browse/EMT-2244
Summary
--> Removed following test apps/folders -
--> Moved Unit tests to BranchSDK.xcodeproj workspace and added app BranchSDKTestsHostApp to run these tests.
--> Updated GHAs for above changes.
Testing Instructions
Source Code Build should be successful.
Unit Test Build and Run should be successful.
All GHA should pass.
cc @BranchMetrics/saas-sdk-devs for visibility.