Skip to content

Convert Blog to Swift#25338

Open
kean wants to merge 2 commits intotrunkfrom
task/convert-blog-to-swift-final
Open

Convert Blog to Swift#25338
kean wants to merge 2 commits intotrunkfrom
task/convert-blog-to-swift-final

Conversation

@kean
Copy link
Contributor

@kean kean commented Mar 3, 2026

@kean kean added this to the 26.8 milestone Mar 3, 2026
@kean kean requested a review from crazytonyli March 3, 2026 18:40
@kean kean added the Tech Debt label Mar 3, 2026
@kean kean force-pushed the task/convert-blog-to-swift-final branch from 536e844 to ff29e1c Compare March 3, 2026 18:42
@wpmobilebot
Copy link
Contributor

wpmobilebot commented Mar 3, 2026

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 Number31395
VersionPR #25338
Bundle IDcom.jetpack.alpha
Commit73bdfa2
Installation URL7bv23dvdcdr3g
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@jkmassel jkmassel removed the Tech Debt label Mar 5, 2026
@kean kean requested a review from jkmassel March 9, 2026 14:58
@dangermattic
Copy link
Collaborator

1 Error
🚫 PR requires at least one label.

Generated by 🚫 Danger

@kean
Copy link
Contributor Author

kean commented Mar 10, 2026

Hey, @crazytonyli or @jkmassel, could you please take a look at this PR when you have time? It's the only change required before WordPressData can be folded into SwiftPM.

@sonarqubecloud
Copy link

@wpmobilebot
Copy link
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 Number31395
VersionPR #25338
Bundle IDorg.wordpress.alpha
Commit73bdfa2
Installation URL75d6j9772mkro
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@wpmobilebot
Copy link
Contributor

🤖 Build Failure Analysis

This build has failures. Claude has analyzed them - check the build annotations for details.

Copy link
Contributor

@crazytonyli crazytonyli left a comment

Choose a reason for hiding this comment

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

🤦‍♂️ I forgot to submit my review

case `private` = -1
case hidden = 0
case `public` = 1
case unknown = 999999
Copy link
Contributor

Choose a reason for hiding this comment

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

The original value of the unknown case is NSIntegerMax.

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.

5 participants