-
Notifications
You must be signed in to change notification settings - Fork 316
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
carthage_installation_tests
: optimize SPM package loading
#3129
Conversation
@@ -7,3 +7,15 @@ desc "Update carthage commit" | |||
backup_extension = '.bck' | |||
sh("sed", '-i', backup_extension, sed_regex, '../Cartfile') | |||
end | |||
|
|||
lane :installation_tests do | |||
load_spm_dependencies |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the core change, adding this which was introduced in #3119.
carthage_installation_tests
: optimize SPM package loadingcarthage_installation_tests
: optimize SPM package loading
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## main #3129 +/- ##
==========================================
+ Coverage 86.60% 86.62% +0.01%
==========================================
Files 219 219
Lines 15713 15713
==========================================
+ Hits 13609 13611 +2
+ Misses 2104 2102 -2 ☔ View full report in Codecov by Sentry. |
ea1da72
to
d9dedb6
Compare
lane :installation_tests do | ||
load_spm_dependencies | ||
|
||
Dir.chdir("..") do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this also use "#{File.dirname(__FILE__)}/../
? I guess the circleci job already has the working_directory
set, so it would probably work without this, but thinking in case we want to run this locally for any reason.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you run it locally you'd have to run it from Tests/InstallationTests/CarthageInstallation
and then doing fastlane installation_tests
. So I think it's fine to leave this as an implicitly relative path.
**This is an automatic release.** ### Bugfixes * `DebugViewModel`: fixed runtime crash on iOS < 16 (#3139) via NachoSoto (@NachoSoto) ### Performance Improvements * `PurchasesOrchestrator`: return early if receipt has no transactions when checking for promo offers (#3123) via Mark Villacampa (@MarkVillacampa) * `Purchases`: don't clear intro eligibility / purchased products cache on first launch (#3067) via NachoSoto (@NachoSoto) ### Dependency Updates * `SPM`: update `Package.resolved` (#3130) via NachoSoto (@NachoSoto) ### Other Changes * `ReceiptParser`: fixed SPM build (#3144) via NachoSoto (@NachoSoto) * `carthage_installation_tests`: optimize SPM package loading (#3129) via NachoSoto (@NachoSoto) * `CI`: add workaround for `Carthage` timing out (#3119) via NachoSoto (@NachoSoto) * `Integration Tests`: workaround to not lose debug logs (#3108) via NachoSoto (@NachoSoto)
Follow up to #3119. This job also needed the workaround to make sure `Carthage` doesn't time out. Also moved the implementation to Fastlane.
**This is an automatic release.** ### Bugfixes * `DebugViewModel`: fixed runtime crash on iOS < 16 (#3139) via NachoSoto (@NachoSoto) ### Performance Improvements * `PurchasesOrchestrator`: return early if receipt has no transactions when checking for promo offers (#3123) via Mark Villacampa (@MarkVillacampa) * `Purchases`: don't clear intro eligibility / purchased products cache on first launch (#3067) via NachoSoto (@NachoSoto) ### Dependency Updates * `SPM`: update `Package.resolved` (#3130) via NachoSoto (@NachoSoto) ### Other Changes * `ReceiptParser`: fixed SPM build (#3144) via NachoSoto (@NachoSoto) * `carthage_installation_tests`: optimize SPM package loading (#3129) via NachoSoto (@NachoSoto) * `CI`: add workaround for `Carthage` timing out (#3119) via NachoSoto (@NachoSoto) * `Integration Tests`: workaround to not lose debug logs (#3108) via NachoSoto (@NachoSoto)
Follow up to #3119. This job also needed the workaround to make sure
Carthage
doesn't time out.Also moved the implementation to Fastlane.