This PR can be summarized in the following changelog entry:
N/A
Relevant technical choices:
Ruleset updates:
Remove dependencies related exclude-pattern. These are now contained within the YoastCS ruleset as of v1.3.0.
Remove the languages directory exclude-pattern.
The languages directory contains no PHP files, so there is no need to explicitly exclude it.
Remove the minimum_supported_wp_versionconfig directive. As of YoastCS 1.3.0, a default is set in the Yoast ruleset and should be followed by this repo.
The default in the Yoast ruleset at this time is 4.9.
Add the yoast_acf prefix (as proscibed for the WordPress.NamingConventions.PrefixAllGlobals sniff) to the list of prefixes which should be removed from filenames.
Minor spelling fix in inline documentation.
Test instructions
This PR can be tested by following these steps:
N/A
This is a code-only change and should have no effect on the functionality.
igorschoester
commented
4 years ago
Summary
This PR can be summarized in the following changelog entry:
Relevant technical choices:
Ruleset updates:
exclude-pattern. These are now contained within the YoastCS ruleset as of v1.3.0.languagesdirectoryexclude-pattern. The languages directory contains no PHP files, so there is no need to explicitly exclude it.minimum_supported_wp_versionconfigdirective. As of YoastCS 1.3.0, a default is set in theYoastruleset and should be followed by this repo. The default in theYoastruleset at this time is4.9.yoast_acfprefix (as proscibed for theWordPress.NamingConventions.PrefixAllGlobalssniff) to the list of prefixes which should be removed from filenames.Test instructions
This PR can be tested by following these steps: