WPCS/I18n refactor breaks HM-Minimum on PHP 8.2

[benjaminprojas]

WPCS recently deprecated a temporary property called check_translator_comments that is currently being used in the HM-Minimum ruleset.

Commit details here: WordPress/WordPress-Coding-Standards@31064cf

Affected code is here:

coding-standards/HM-Minimum/ruleset.xml

Lines 160 to 163 in e8ec095

	<rule ref="WordPress.WP.I18n">
	<properties>
	<property name="check_translator_comments" value="false" />
	</properties>

I think the solution is as easy as:

<rule ref="WordPress.WP.I18n">
	<exclude name="WordPress.WP.I18n.MissingTranslatorsComment" />
	...

[rmccue]

Thanks! I've fixed this in #314 as part of our phpcs upgrade.

[benjaminprojas]

@rmccue that's great to hear! We ended up forking this repo and bringing it all up to speed for PHP 8.2 as I didn't think it was being maintained anymore. Excited to see these updates come through 😄

[rmccue]

We use it every day on many projects, we've just been slow to update to PHP 8.2 due to WP's compatibility with it :)