Corey Farrell

@JoshMcCullough Yes the code works, note that the parameters for my replacement function are `(warning, ...args)`. I can see how that looks odd but it definitely works.

https://nodejs.org/dist/latest/docs/api/process.html#process_process_emitwarning_warning_options shows that process.emitWarning can also take an options object as the second argument. I don't think the ESM warning uses that style call but I still check for it...

If I make changes to index.js and run `xo` without running `git add index.js` then xo could end up modifying my unstaged file. If this were done I would add...

Personally I use "consistent" array-bracket-newline, wish I could do the same with array-element-newline. Just leave it to maximum line length to enforce line feeds between array/object elements, but enforce that...

Unrelated note while inspecting my global install I noticed some `.DS_Store` files leaked into the npm publish of `xo` and related modules.

@rizowski Thanks for the suggestion however I just ran `npm i -g [email protected]` and still have the issue. I verified the correct version was installed globally.

This is reasonable, thanks for responding. Up to you if you want to close this issue or leave it open so users can potentially see it and test on their...

What about `throw`? Examples: ```js function f1() { if (true) { throw new Error('throw'); } return 'unreachable'; } function f2() { if (true) { throw new Error('throw'); } else {...

> Add a `package.json` inside `lib` with its own "exports". To my knowledge "exports" is only applied from the package.json at the top-level of any package. I've experimented with putting...

I've tried using `internal: Button press and release` to accomplish this and it usually works. The problem I've had is when the target button uses the action `internal: Set surface...