silverwind

I don't understand rollup in such detail, but do you mean if any plugin in the chain returns `map: { mappings: '' }`, rollup will skip the (computationally intensive) sourcemap...

I also have a case of `--watch` getting stuck since v6.0.0 (at CSS transformation) and can confirm that `build.rollupOptions.maxParallelFileOps: 100` fixes it. There are more than 100 files in my...

Here is the relevant rollup change: https://github.com/rollup/rollup/pull/5986 Note that it just changed the default of `maxParallelFileOps`.

> Or, a bit more ambitiously, add a top level sourcemap setting that, if set, would supercede build.sourcemap, esbuild.sourcemap, build.rollupOptions.output.sourcemap* and css.devSourcemap A single top-level sourcemap option to control all...

> --show-stderr is not possible I think, because the test2json output that gotestsum receives from go test doesn't distinguish stdout from stderr, it's all just "output" Maybe it's possible to...

Also noticed this. Contribution is attributed to the pusher, not the actual author (which is what one usually wants). I often push changes via a bot account and these don't...

> How about `pip install --user`? Can still cause problems with other packages installed as `--user`. The reason why I recommend `pipx` is because it's being packaged by many distributors,...

This is hitting on code like this: ```ts const onKeydown = useCallback((event) => { console.log(event); }, []); useEffect(() => { window.addEventListener("keydown", onKeydown); return () => { window.removeEventListener("keydown", onKeydown); }; },...

There's nothing wrong with the rule, it's doing it's job fine. The fixed code also works as expected. I think I will get used to writing inline functions inside `useEffect`....

The fixed variant is also how react's docs [recommend](https://react.dev/learn/synchronizing-with-effects#subscribing-to-events) using `addEventListener`, so it is definitely the right way. So I'm happy with the rule as-is and I think I don't...