【 2 + 1 Suggestions】The eslint config of PIXI.js is too much strict .

One : “no-unmodified-loop-condition”

I have a function like this :

funcA(loopMoreThanOnce)
{
    //  .....
    do {
        //  .....
    } while (loopMoreThanOnce);
    //  .....
}

This funcA is reasonable. But because "no-unmodified-loop-condition": 1 , I have to write the code very ugly like this :

funcA(loopMoreThanOnce)
{
    //  .....
    let flag;
    do {
        flag = loopMoreThanOnce;
        //  .....
    } while (flag);
    //  .....
}

Could you remove it please ?

Two : “func-names”

Because of "func-names": [1, "always"]

When I want to define a function have to write like this :

const funcName = function funcName()
{

}

The name on right side is unnecessary and stupid.

Could you remove it please ?

Three : “quote-props” & “dot-notation”

I want to use 'key' for the same object , but because "quote-props": [1, "as-needed"] & "dot-notation" : 1 , I can’t use Uniform Style in my code：

// look at the keys : 
        this.compositeOperations = {
            'source-over': [gl.ONE, gl.ONE_MINUS_SRC_ALPHA, 1],
            lighter: [gl.ONE, gl.ONE_MINUS_SRC_ALPHA, 0],
            lighten: [gl.ONE, gl.ONE_MINUS_SRC_ALPHA, 0],
            darker: [gl.DST_COLOR, gl.ONE_MINUS_SRC_ALPHA, 1],
            darken: [gl.DST_COLOR, gl.ONE_MINUS_SRC_ALPHA, 1],
            'destination-out': [gl.ZERO, gl.ONE_MINUS_SRC_ALPHA, 1],
            'destination-over': [gl.ONE_MINUS_DST_ALPHA, gl.ONE, 1],
            'source-atop': [gl.DST_ALPHA, gl.ONE_MINUS_SRC_ALPHA, 1],
            xor: [gl.ONE_MINUS_DST_ALPHA, gl.ONE_MINUS_SRC_ALPHA, 1],
            copy: [gl.ONE, gl.ZERO, 1],
            'source-in': [gl.DST_ALPHA, gl.ZERO, 1],
            'destination-in': [gl.ZERO, gl.SRC_ALPHA, 1],
            'source-out': [gl.ONE_MINUS_DST_ALPHA, gl.ZERO, 1],
            'destination-atop': [gl.ONE_MINUS_DST_ALPHA, gl.SRC_ALPHA, 1],
        };

obj['source-over'];
obj.lighter;
obj.lighten;
obj.darker;
obj.darken;
obj['destination-out'];
obj['destination-over'];
obj['source-atop'];
obj.xor;
obj.copy;
obj['source-in'];
obj['destination-in'];
obj['source-out'];
obj['destination-atop'];

[themoonrat]

Hi finscn. Sorry you don’t like it, but as BossLevel, it’s an extra method name for the sake of extra debugging information. I’m sure you can understand why, given a choice, developers of a library might choose ease of debugging over more concise syntax.

[lock[bot]]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.