Skip to content

Build/Test Tools: Declare dealerdirect/phpcodesniffer-composer-installer explicitly - #72

Draft
mukeshpanchal27 wants to merge 1 commit into
trunkfrom
deps/declare-phpcs-composer-installer
Draft

Build/Test Tools: Declare dealerdirect/phpcodesniffer-composer-installer explicitly#72
mukeshpanchal27 wants to merge 1 commit into
trunkfrom
deps/declare-phpcs-composer-installer

Conversation

@mukeshpanchal27

Copy link
Copy Markdown
Owner

Problem

dealerdirect/phpcodesniffer-composer-installer is listed in config.allow-plugins,
but is not in require-dev. It arrives transitively via wp-coding-standards/wpcs and
phpcompatibility/phpcompatibility-wp.

That plugin is what populates the PHP_CodeSniffer installed_paths setting — i.e. whether
composer lint can resolve the WordPress standard at all depends on a package nobody
declared. The project already explicitly trusts it in allow-plugins, so the dependency
is deliberate; only the declaration is missing.

Change

Adds the explicit require-dev entry so the tool that makes linting work is visible.


Draft proposal from a review of composer.json / package.json. Opened as a draft for discussion — not intended to merge as-is.

…ler explicitly

See the PR description for the full rationale and evidence.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant