Automattic / woocommerce-subscriptions-core

Subscriptions core package for WooCommerce
Other
85 stars 31 forks source link

Fix PHPCS violations in `WCS_Admin_System_Status` to improve code quality #387

Closed Jinksi closed 1 year ago

Jinksi commented 1 year ago

Based on #386. Please wait for it to be merged before reviewing/merging.

Description

This PR only fixes PHPCS violations in the WCS_Admin_System_Status class to improve code quality. No changes to the underlying logic are expected.

This class is used to add details to WooCommerce → Status.

How to test this PR

  1. Run ./vendor/bin/phpcs includes/admin/class-wcs-admin-system-status.php in your terminal. On trunk you should see errors/warnings. On this branch, you should not see any violations. Note: running phpcs directly from ./vendor/bin/ skips the "only changed LOC" rule that we use for PHPCS GH checks.

Product impact