Automattic / Automattic/VIP-Coding-Standards

Almost every WP CLI command should call `stop_the_insanity()`

Open
#151 2 comments 0 reactions 0 assignees View on GitHub
Type: Enhancement
Dominant language
PHP
Stars
261
Forks
44
Avg merge
19m
Merged PRs (30d)
1

Description

see https://vip.wordpress.com/documentation/writing-bin-scripts/ for more details.

We, imho, should produce a warning in case the `stop_the_insanity()` is not part of a command.

Contributor guide

Open the contributing guide

Research direction

Start with the linked VIP documentation on writing bin scripts, then inspect the repository's PHP_CodeSniffer rules for WordPress CLI commands. Done means commands missing stop_the_insanity() produce the requested warning, with coverage for the relevant command cases.

Written by the indexing model from the issue text.

Assessment

Tech stack
php, wordpress
Domain
cli, tooling
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
28/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.