-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Update dependency fastlane-plugin-wpmreleasetoolkit to v11 #20922
Update dependency fastlane-plugin-wpmreleasetoolkit to v11 #20922
Conversation
Generated by 🚫 Danger |
📲 You can test the changes from this Pull Request in WordPress by scanning the QR code below to install the corresponding build.
|
📲 You can test the changes from this Pull Request in Jetpack by scanning the QR code below to install the corresponding build.
|
@iangmaia @AliSoftware - is it safe to merge? I'd like to double-check with you, as this is a bump of 2 major. |
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 major bump can't be merged without changes to the fastlane files (see inline comment + migration guide)
gem 'fastlane-plugin-wpmreleasetoolkit', '~> 9.2' | ||
gem 'fastlane-plugin-wpmreleasetoolkit', '~> 11.0' |
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.
See https://github.com/wordpress-mobile/release-toolkit/blob/trunk/MIGRATION.md
- Project uses ruby version
3.2.2
➡️ yes -
setbranchprotection
renamed toset_branch_protection
, withcopy_branch_protection(from:…)
being usually even a better replacement for it ➡️ ❌ Used infastlane/lanes/release.rb:79
-
removebranchprotection
renamedremove_branch_protection
➡️ ❌ Used infastlane/lanes/release.rb:296
-
setfrozentag
renamedset_milestone_frozen_marker
➡️ ❌ Used infastlane/lanes/release.rb:80
and infastlane/lanes/release.rb:318
-
: Not applicable for Android projectsios_clear_intermediate_tags
andios_final_tag
actions removed -
has_alpha_version
removed ➡️ ✅ Not used -
project_name
andproject_root_folder
removed ➡️ ✅ Not used -
update_pull_requests_milestone
renamed toupdate_assigned_milestone
(+ parameters renamed) ➡️ ✅ Not used -
: N/A to Android projectsios_check_beta_deps
parameter updated
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.
I've updated the calls to be in line with the release-toolkit
version 11.0.2
(cc @wzieba)
479189c
to
d20a7bf
Compare
fastlane/lanes/release.rb
Outdated
@@ -76,8 +76,8 @@ | |||
|
|||
push_to_git_remote(tags: false) | |||
|
|||
setbranchprotection(repository: GHHELPER_REPO, branch: "release/#{new_version}") | |||
setfrozentag(repository: GHHELPER_REPO, milestone: new_version) | |||
set_branch_protection(repository: GHHELPER_REPO, branch: "release/#{new_version}") |
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.
Shouldn't we start using copy_branch_protection(from: DEFAULT_BRANCH, to: "release/…")
instead?
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.
At first I thought it would be ok to keep as set, but I realize now apps seem to use copy_branch_protection
; also, it will just follow whatever we have in trunk
, which might be different per repo so it seems more consistent. Changed on a378a5e.
Quality Gate passedIssues Measures |
This PR contains the following updates:
'~> 9.2'
->'~> 11.0'
Release Notes
wordpress-mobile/release-toolkit (fastlane-plugin-wpmreleasetoolkit)
v11.0.1
Compare Source
Bug Fixes
android_download_translations
action by correctly calling Fastlane'sgit_submodule_update
action [#561]v11.0.0
Compare Source
Breaking Changes
ios_check_beta_deps
use thePodfile.lock
instead ofPodfile
for its detection, and also be able to detect Pods referenced by commits and branches.If your
Fastfile
called this action with an explicitpodfile: …
argument, you'll have to update the call to uselockfile:
instead (or rely on defaults). [#557]v10.0.0
Compare Source
Breaking Changes
3.2.2
. [#517]setbranchprotection
andremovebranchprotection
backwards-compatiblity stubs for the now-renamedset_branch_protection
andremove_branch_protection
actions. [#549]setfrozentag
action toset_milestone_frozen_marker
. [#548]ios_clear_intermediate_tags
action, which has been deprecated for a while. [#549]has_alpha_version
option from several actions and helper methods. It has already been deprecated for many versions. [#550]project_name
andproject_root_folder
options from several actions. [#550]update_pull_requests_milestone
toupdate_assigned_milestone
and make it handle GitHub issues as well as PRs. [#547]Bug Fixes
comment_on_pr
to allow first paragraph of the comment to still be interpreted as Markdown. [#544]Internal Changes
GitHelper.ensure_on_branch!
method [#531]GitHelper.update_submodules
method [#531]nokogiri
,mini_portile2
, andrmagick
[#546]Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR has been generated by Mend Renovate. View repository job log here.