Skip to content

Fix: Add missing strict_types declarations to Abilities Explorer#208

Merged
dkotter merged 1 commit intoWordPress:developfrom
huzaifaalmesbah:fix/strict-types-abilities-explorer
Feb 2, 2026
Merged

Fix: Add missing strict_types declarations to Abilities Explorer#208
dkotter merged 1 commit intoWordPress:developfrom
huzaifaalmesbah:fix/strict-types-abilities-explorer

Conversation

@huzaifaalmesbah
Copy link
Member

@huzaifaalmesbah huzaifaalmesbah commented Feb 2, 2026

Description

This PR standardizes the code in the includes/Experiments/Abilities_Explorer directory by adding: declare( strict_types=1 ); to files where it was missing.

Changes

Added declare( strict_types=1 ); to:

  • includes/Experiments/Abilities_Explorer/Abilities_Explorer.php
  • includes/Experiments/Abilities_Explorer/Ability_Handler.php
  • includes/Experiments/Abilities_Explorer/Ability_Table.php
  • includes/Experiments/Abilities_Explorer/Admin_Page.php

Reasoning

  • Ensures strict type enforcement
  • Improves type safety and reduces implicit type coercion bugs
  • Aligns these files with the rest of the codebase (e.g., includes/bootstrap.php, includes/helpers.php)
  • Maintains consistency and coding standards across the project
Open WordPress Playground Preview
@github-actions
Copy link

github-actions bot commented Feb 2, 2026

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: huzaifaalmesbah <huzaifaalmesbah@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

codecov bot commented Feb 2, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 50.80%. Comparing base (b8c3e49) to head (1351969).
⚠️ Report is 5 commits behind head on develop.

Additional details and impacted files
@@            Coverage Diff             @@
##             develop     #208   +/-   ##
==========================================
  Coverage      50.80%   50.80%           
  Complexity       375      375           
==========================================
  Files             27       27           
  Lines           1978     1978           
==========================================
  Hits            1005     1005           
  Misses           973      973           
Flag Coverage Δ
unit 50.80% <ø> (ø)

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.
@jeffpaul jeffpaul requested a review from dkotter February 2, 2026 18:41
@jeffpaul jeffpaul added this to the 0.3.0 milestone Feb 2, 2026
@jeffpaul jeffpaul moved this to Needs review in WordPress AI Planning & Roadmap Feb 2, 2026
@dkotter dkotter merged commit 2cfe713 into WordPress:develop Feb 2, 2026
24 checks passed
@github-project-automation github-project-automation bot moved this from Needs review to Done in WordPress AI Planning & Roadmap Feb 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants