Loading...

XML

Word

Printable

Some versions ago we added a new sniff to look for proper spacing around operators:

Squiz.WhiteSpace.OperatorSpacing

with tests:

In the plugin DB, gb2048 commented that such a sniff is missing some operators (bitwise and friends):

Gents,

Using 2.3.1 - San Fermín release! and getting errors for legitimate code in a js file:

value·=·value·<<·2;

Expected 1 space after "<"; 0 found

Expected 1 space before "<"; 0 found

.....

value·|=·toggleflag;

Expected 1 space after "|"; 0 found

Expected 1 space before "="; 0 found

.....

value·&=·~toggleflag;

Expected 1 space after "&" operator; 0 found

Expected 1 space before "="; 0 found

This is about to add those missing operators to the Sniff in order to get them properly evaluated.

Ciao

has a non-specific relationship to

MDL-52127 Check Moodle JS coding style with eslint

will be (partly) resolved by

CONTRIB-6175 Make codechecker to ignore JS files completely

CONTRIB-8122 Undue error report in codechecher (processing JS files)