Daniel Cousens

Results 530 comments of Daniel Cousens
trafficstars

@jprichardson > // magic function blackList is defined somewhere... https://github.com/dcousens/blacklist, may or may not fit the syntax you want

@LinusU the other major places I had this issue (`superagent`) were eventually lobbied enough to remove it, but I still feel its pain from time to time.

Indeed, I am bitten by this often. The few times it still occurs in `express`, I use `_` to indicate an unused variable.

What about ``` json "no-unused-vars": [2, { "vars": "all", "args": "all", "argsIgnorePattern": "^_$" }], ```

I understand the ecosystem impact is the blocker, but, `_` signals the difference between "forgotten to use" and "intentionally omitted"

This error is almost impossible to miss, as the code simply will fail at `require` time in almost all cases. I'm OK with it being disabled.

I merged that change, but, we can roll it back if necesssary. If `h` is undefined, then, you should `global` it out IMHO.

@feross perhaps we ditch `react` in the next major? Is it possible for us to do things like `eslint-env react`?