Skip to content

Conversation

@RafaelKayumov
Copy link
Contributor

@RafaelKayumov RafaelKayumov commented Nov 24, 2025

Description

As discussed: p91TBi-dwL-p2#comment-14434 - removing the Periphery from Buildkite pipeline.

  • Remove the "Detect unused code" step from .buildkite/pipeline.yml
  • Remove DerivedData/Index.noindex/DataStore/export from .buildkite/commands/build-for-testing.sh
  • Keeps run / setup scripts for Periphery for future re-use.

Test Steps

Just make sure the buildkite pipeline works fine and there is no Detect unused code step in output log.


  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

@RafaelKayumov RafaelKayumov added this to the 23.8 milestone Nov 24, 2025
@RafaelKayumov RafaelKayumov added the category: tooling Anything that involves building & maintaining the project, including scripts, `Fastfile`, etc. label Nov 24, 2025
@wpmobilebot
Copy link
Collaborator

App Icon📲 You can test the changes from this Pull Request in WooCommerce iOS Prototype by scanning the QR code below to install the corresponding build.

App NameWooCommerce iOS Prototype
Build Numberpr16387-c0e1576
Version23.7
Bundle IDcom.automattic.alpha.woocommerce
Commitc0e1576
Installation URL0r68q13nh3vmg
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@RafaelKayumov
Copy link
Contributor Author

CC @AliSoftware

Copy link
Contributor

@AliSoftware AliSoftware left a comment

Choose a reason for hiding this comment

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

👍

PS: Don't hesitate to ping AppsInfra when you'll start on making those run on scheduled jobs instead.

If you feel inclined, feel free to create the .buildkite/run-periphery.yml pipeline yourself with the necessary steps (to build + call run-periphery.sh) and test it via manually-triggered builds on Buildkite (setting PIPELINE=run-periphery.yml in the options dropdown after clicking Manual Build), then we can help in creating a schedule to run that pipeline at specific times in our infra.

@RafaelKayumov RafaelKayumov merged commit 3c78cc7 into trunk Nov 25, 2025
18 of 19 checks passed
@RafaelKayumov RafaelKayumov deleted the WOOMOB-1761-remove-periphery-step-from-buildkite-pipeline branch November 25, 2025 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: tooling Anything that involves building & maintaining the project, including scripts, `Fastfile`, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants