printfn
printfn
Rebased and fixed merge conflicts. Would someone be able to review this?
I'm using biome on a large legacy codebase, slowly migrating from ESLint. Having the option to enable `noUnusedVariables` would be super helpful in improving code quality, but there are *thousands*...
It's also worth pointing out that there are many repos on GitHub that set ESLint's `args` to `none`: https://github.com/search?q=%2F%22no-unused-vars%22%3A%5Cs*%5C%5B%5Cs*%22%28warn%7Cerror%29%22%2C%5Cs*%5C%7B.*%22args%22%3A%5Cs*%22none%22.*%5C%7D%5C%5D%2F&type=code
I'll give making a new `noUnusedFunctionParameters` rule a go.
Alright, pretty sure I got it all working now. I've added a new nursery lint `noUnusedFunctionParameters` and changed `noUnusedVariables` to no longer include function parameters.
this is ready for another review @ematipico
Another workaround is `c/(145MHz) to m`. But yes, this is definitely an issue that should be fixed.
Sure, but I think it will be a bit tricky to fix. This issue is the result of two separate bugs in fend: on one hand, fend's operator precedence rules...
I've pushed a change that implements better unit simplification, which resolves part of this issue. Units like `m s^-1 MHz^-1` now get simplified to `meters`. ``` > c / (145MHz)...
Thanks for the suggestion, I agree this would be a nice feature :)