Deprecate Deliverfile in favor of Fastfile#1084
Conversation
AliSoftware
left a comment
There was a problem hiding this comment.
Similar to woocommerce/woocommerce-ios#8831 (review) and Automattic/simplenote-ios#1515 (review), we need to add skip_deliver: true to the call site which will try to bump the version from the Deliverfile now that that file is gone
(this is something we also forgot to do the first time we did such a change in WPiOS… which required a follow-up commit to fix it later…)
Good points, and same answers as the ones I gave in Automattic/simplenote-ios#1515 (comment):
|
Fix
Similarly to what was done for WPiOS on wordpress-mobile/WordPress-iOS#16805, this PR removes the
Deliverfilemoving everything to a new lane in theFastfile. For that I also moved out theprivacy_urland thecopyrightto separate files in themetadatafolder.Additionally, the need for an
app_versionset in theDeliverfilehas been deprecated and its usage is gonna soon be removed altogether from release-toolkit, so this PR already cleans this up.Test
Testing the new lane is similar to wordpress-mobile/WordPress-iOS#16805 (and is particularly difficult for me to do completely, so I'd also appreciate some help):
Also good to test the use of the parameter
with_screenshots:trueto see that it will attempt to load the screenshots.Review
I have noticed that this project uses an older version of the release-toolkit, so perhaps this upgrade needs to happen before this PR lands.
Release
These changes do not require release notes.