PHPCS config: exclude plugins and themes within src

[johnbillion]

When running the PHPCS linting locally (eg. with composer lint:errors), there are files that can exist within an installation that's running from src which should not be linted:

• Plugins
• Must-Use Plugins
• Drop-in plugins
• Themes other than the twenty* themes

These should all be excluded from linting so that a clean lint can be run on an active installation of WordPress running from src.

[jrf]

@johnbillion How do those files get into the src directory ?
Are there dev environment commands in place which generate those files ? or put them there ?

If not and if it's dev choice to put them there (for testing purposes or otherwise), this could become a never-ending list as different devs use different tools.

In that case, adjusting the WP Core ruleset is not the way to "fix" this.
Instead you should add a phpcs.xml file to the project root in your local environment. That file will be automatically ignored for committing, but should be automatically picked up by the PHPCS run.

In that file, you'd then have something like the following:

<?xml version="1.0"?>
<ruleset name="My WP Core ruleset">

	<!-- Import the main ruleset. -->
	<rule ref="./phpcs.xml.dist"/>

	<!-- ... add your own exclude rules tailored to your dev environment ... -->

</ruleset>

Does that help ?

[johnbillion]

This is for running a WordPress site from src using npm run dev. It's a standard setup.

None of the exclusions are custom for my particular dev environment, they're all files that WordPress core supports and uses.

[johnbillion]

It's worth noting that all the files listed in 49781.diff​ already exist in .gitignore for the same reason.

[jrf]

Ok, in that case, sounds reasonable. Though it might be helpful to annotate the start and end of the list with a note saying that that list should be kept in sync with the .gitignore file ?

[johnbillion]

That's a good idea, and reminds me that the svn:ignore prop for SVN needs to be synced too as it's out of date. I'll open a ticket.

[johnbillion]

In 47742:

Build/Test Tools: Exclude plugins and themes within src.

Just like the build directory, src can include plugins and themes which shouldn't be linted by PHPCS.

Fixes #49781

[desrosj]

Referenced in the following dev note: ​https://make.wordpress.org/core/2020/07/14/php-related-improvements-changes-wordpress-5-5-edition/.