Skip to content

Remove ios and android build_preflight - #773

Merged
mokagio merged 5 commits into
trunkfrom
ainfra-2960-remove-ios_build_preflight-from-release-toolkit-after
Aug 26, 2026
Merged

Remove ios and android build_preflight#773
mokagio merged 5 commits into
trunkfrom
ainfra-2960-remove-ios_build_preflight-from-release-toolkit-after

Conversation

@mokagio

@mokagio mokagio commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

What does it do?

Remove the ios_build_preflight and android_build_preflight actions.

Android callers have already landed.
iOS callers still have unmerged PRs: WordPress-iOS#25934, woocommerce-ios#17759.

Breaking change for 15.0.0 (AINFRA-2961).

Checklist before requesting a review

  • Run bundle exec rubocop to test for code style violations and recommendations.
  • Add Unit Tests (aka specs/*_spec.rb) if applicable. None — neither action had a spec.
  • Run bundle exec rspec to run the whole test suite and ensure all your tests pass. 1007 examples, 0 failures.
  • Make sure you added an entry in the CHANGELOG.md file to describe your changes under the appropriate existing ### subsection of the existing ## Trunk section.
  • If applicable, add an entry in the MIGRATION.md file to describe how the changes will affect the migration from the previous major version and what the clients will need to change and consider.

The last remaining callers are WordPress-iOS and WooCommerce iOS,
each with an approved PR to drop the call.
The action ran `configure_apply`, which will become a no-op or a
failure once we move to `a8c-secrets`.

---

Generated with the help of Grok, https://x.ai/grok

Co-Authored-By: Grok 4.6 <noreply@x.ai>
@mokagio
mokagio requested a review from a team as a code owner August 26, 2026 03:53
Copilot AI lite review requested due to automatic review settings August 26, 2026 03:53
@mokagio mokagio self-assigned this Aug 26, 2026
---

Generated with the help of Grok, https://x.ai/grok

Co-Authored-By: Grok 4.6 <noreply@x.ai>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request removes the deprecated ios_build_preflight Fastlane action from the Release Toolkit gem, and documents the breaking change for the upcoming 15.0.0 release so downstream clients can migrate cleanly.

Changes:

  • Remove the ios_build_preflight iOS action implementation.
  • Add migration guidance for consumers upgrading to 15.0.0.
  • Add a Breaking Changes entry to the CHANGELOG for ios_build_preflight removal.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
lib/fastlane/plugin/wpmreleasetoolkit/actions/ios/ios_build_preflight.rb Deletes the ios_build_preflight action implementation.
MIGRATION.md Documents the required migration steps for 14.x → 15.0.0 regarding this removal.
CHANGELOG.md Records the breaking change under ## Trunk so it’s surfaced in release notes.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread CHANGELOG.md Outdated
Comment thread MIGRATION.md Outdated
mokagio and others added 2 commits August 26, 2026 13:55
---

Generated with the help of Grok, https://x.ai/grok

Co-Authored-By: Grok 4.6 <noreply@x.ai>
All four client removals have landed.
Same breaking change as the ios_build_preflight deletion.

---

Generated with the help of Grok, https://x.ai/grok

Co-Authored-By: Grok 4.6 <noreply@x.ai>
@mokagio mokagio changed the title Remove ios_build_preflight action Remove ios and android build_preflight Aug 26, 2026
@mokagio
mokagio enabled auto-merge August 26, 2026 04:06
Comment thread MIGRATION.md Outdated

@iangmaia iangmaia left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving after a small change in MIGRATION.md, leaving auto-merge on to avoid back and forth.

@mokagio
mokagio merged commit fcedf25 into trunk Aug 26, 2026
6 checks passed
@mokagio
mokagio deleted the ainfra-2960-remove-ios_build_preflight-from-release-toolkit-after branch August 26, 2026 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants