Skip to content

Remove unused WPDATA preprocessor macro - #24517

Merged
mokagio merged 1 commit into
trunkfrom
mokagio/remove-wpdata-preprocessor
May 7, 2025
Merged

Remove unused WPDATA preprocessor macro#24517
mokagio merged 1 commit into
trunkfrom
mokagio/remove-wpdata-preprocessor

Conversation

@mokagio

@mokagio mokagio commented May 5, 2025

Copy link
Copy Markdown
Contributor

It was part of the scaffold to introduce WordPressData but is no longer necessary.

To verify removing it caused no issues, see green CI build and the text searches in the screenshot below:

image

image

It was part of the scaffold to introduce WordPressData but is no longer
necessary.
@sonarqubecloud

sonarqubecloud Bot commented May 5, 2025

Copy link
Copy Markdown

@wpmobilebot

Copy link
Copy Markdown
Contributor
App Icon📲 You can test the changes from this Pull Request in Jetpack by scanning the QR code below to install the corresponding build.
App NameJetpack
ConfigurationRelease-Alpha
Build Number27579
VersionPR #24517
Bundle IDcom.jetpack.alpha
Commit3b7f0f8
Installation URL0u108u5hdqbrg
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@wpmobilebot

Copy link
Copy Markdown
Contributor
App Icon📲 You can test the changes from this Pull Request in WordPress by scanning the QR code below to install the corresponding build.
App NameWordPress
ConfigurationRelease-Alpha
Build Number27579
VersionPR #24517
Bundle IDorg.wordpress.alpha
Commit3b7f0f8
Installation URL76g7f6stsrr3o
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@mokagio mokagio added this to the 26.1 milestone May 5, 2025
@mokagio mokagio self-assigned this May 5, 2025
@mokagio
mokagio requested review from crazytonyli and kean May 6, 2025 04:32
@mokagio
mokagio added this pull request to the merge queue May 7, 2025
Merged via the queue into trunk with commit 2d9d089 May 7, 2025
@mokagio
mokagio deleted the mokagio/remove-wpdata-preprocessor branch May 7, 2025 04: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