Skip to content

Convert ReaderPost mapping to Swift#25301

Open
kean wants to merge 5 commits intotrunkfrom
task/convert-remote-post-mapper
Open

Convert ReaderPost mapping to Swift#25301
kean wants to merge 5 commits intotrunkfrom
task/convert-remote-post-mapper

Conversation

@kean
Copy link
Contributor

@kean kean commented Feb 25, 2026

I used TDD to reduce the risk of regression and performed manual tests.

@kean kean added the Tech Debt label Feb 25, 2026
@kean kean added this to the Someday milestone Feb 25, 2026
@dangermattic
Copy link
Collaborator

dangermattic commented Feb 25, 2026

2 Warnings
⚠️ Modules/Package.swift was changed without updating its corresponding Package.resolved.

If the change includes adding, removing, or editing a dependency please resolve the Swift packages as appropriate to your project setup (e.g. in Xcode or by running swift package resolve).

If the change to the Package.swift did not modify dependencies, ignoring this warning should be safe, but we recommend double checking and running the package resolution just in case.
.

⚠️ This PR is larger than 500 lines of changes. Please consider splitting it into smaller PRs for easier and faster reviews.

Generated by 🚫 Danger

@wpmobilebot
Copy link
Contributor

wpmobilebot commented Feb 25, 2026

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 Number31134
VersionPR #25301
Bundle IDorg.wordpress.alpha
Commit5015185
Installation URL4iqjafnh1mpu8
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@wpmobilebot
Copy link
Contributor

wpmobilebot commented Feb 25, 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 Number31134
VersionPR #25301
Bundle IDcom.jetpack.alpha
Commit5015185
Installation URL5a6aj06k08oc8
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@kean kean requested review from crazytonyli and jkmassel February 25, 2026 14:41
@wpmobilebot
Copy link
Contributor

wpmobilebot commented Feb 25, 2026

🤖 Build Failure Analysis

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

@sonarqubecloud
Copy link

@kean kean enabled auto-merge February 25, 2026 17:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants