Page MenuHomePhabricator

Add sniff to ensure use of spaces (not tabs) between variable and assignment operator
Open, Needs TriagePublic

Description

From https://gerrit.wikimedia.org/r/534931:

function foo() {
	$aaac = escapeshellarg();
	$bbb»= escapeshellarg();
	    ^ problem
	$ccccd = escapeshellarg();
}

Event Timeline

Krinkle created this task.Sep 8 2019, 1:30 AM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptSep 8 2019, 1:30 AM
Daimona added a subscriber: Daimona.

AFAICS, the rules enforcing spaces before/after operands are in Squiz.WhiteSpace.OperatorSpacing, which means that they should be changed upstream.