Skip to content

Address a few warnings in WordPressData #24528

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
May 8, 2025
Merged

Conversation

mokagio
Copy link
Contributor

@mokagio mokagio commented May 8, 2025

What is says in the title. Noticed them while working on #24518

@mokagio mokagio added this to the 26.1 milestone May 8, 2025
@mokagio mokagio self-assigned this May 8, 2025
@mokagio mokagio requested review from crazytonyli and kean May 8, 2025 06:46
Comment on lines -350 to +352
title = [NSString stringWithFormat:@"%d %@", count, likeStr];
title = [NSString stringWithFormat:@"%ld %@", (long)count, likeStr];
} else {
title = [NSString stringWithFormat:@"%d %@", count, likesStr];
title = [NSString stringWithFormat:@"%ld %@", (long)count, likesStr];
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Xcode suggested this: to cast the NSInteger to long and use it via ld in the formatting.

Copy link

sonarqubecloud bot commented May 8, 2025

@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 Number27623
VersionPR #24528
Bundle IDorg.wordpress.alpha
Commite0de6d9
Installation URL1268hh6c5caf8
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@wpmobilebot
Copy link
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 Number27623
VersionPR #24528
Bundle IDcom.jetpack.alpha
Commite0de6d9
Installation URL7f6avs79k358o
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@mokagio mokagio enabled auto-merge May 8, 2025 10:32
@mokagio mokagio added this pull request to the merge queue May 8, 2025
@kean kean modified the milestones: 26.1, 26.0 May 8, 2025
@dangermattic
Copy link
Collaborator

1 Warning
⚠️ This PR is assigned to the milestone 26.0. The due date for this milestone has already passed.
Please assign it to a milestone with a later deadline or check whether the release for this milestone has already been finished.

Generated by 🚫 Danger

Merged via the queue into trunk with commit 961b6a9 May 8, 2025
37 of 39 checks passed
@mokagio mokagio deleted the mokagio/address-wpdata-warnings branch May 8, 2025 14:26
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.

4 participants