Deprecate Deliverfile
in favor of Fastfile
#1084
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fix
Similarly to what was done for WPiOS on wordpress-mobile/WordPress-iOS#16805, this PR removes the
Deliverfile
moving everything to a new lane in theFastfile
. For that I also moved out theprivacy_url
and thecopyright
to separate files in themetadata
folder.Additionally, the need for an
app_version
set in theDeliverfile
has 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:true
to 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.