Skip to content
This repository was archived by the owner on Feb 5, 2026. It is now read-only.

Fix incomplete translator comments in WP_Ability and WP_Ability_Category#157

Open
igorradovanov wants to merge 1 commit into
WordPress:trunkfrom
igorradovanov:trunk
Open

Fix incomplete translator comments in WP_Ability and WP_Ability_Category#157
igorradovanov wants to merge 1 commit into
WordPress:trunkfrom
igorradovanov:trunk

Conversation

@igorradovanov
Copy link
Copy Markdown

The translator comments in WP_Ability and WP_Ability_Category only describe the first placeholder but the strings have three.

WP_Ability (class-wp-ability.php)

Before:

/* translators: %s: Property name. */

After:

/* translators: %1$s: Property name, %2$s: Ability name, %3$s: Class name. */

WP_Ability_Category (class-wp-ability-category.php)

Before:

/* translators: %s: Property name. */

After:

/* translators: %1$s: Property name, %2$s: Ability category slug, %3$s: Class name. */

Translators using tools like translate.wordpress.org don't see the source code, so having all placeholders described helps them know what each one is.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Dec 30, 2025

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: igorradovanov <igorradovanov@git.wordpress.org>
Co-authored-by: justlevine <justlevine@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 30, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 22.52%. Comparing base (5f64910) to head (8fed9a7).

Additional details and impacted files
@@              Coverage Diff              @@
##              trunk     #157       +/-   ##
=============================================
- Coverage     80.46%   22.52%   -57.94%     
  Complexity      178      178               
=============================================
  Files            20       20               
  Lines          1474     1474               
  Branches        120      119        -1     
=============================================
- Hits           1186      332      -854     
- Misses          288     1142      +854     
Flag Coverage Δ
javascript 94.61% <ø> (ø)
unit 1.40% <ø> (-74.92%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
@justlevine
Copy link
Copy Markdown
Contributor

(Clarifying - approving as this is indeed a docs bug + valid fix but I believe it needs to first be directly applied to wordpress-develop ).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

2 participants