I’m curious if there’s interest in a nixpkgs-fmt bot similar to rmcgibbo’s lovely nixpkgs-review bot?
It doesn’t seem like it’d be that hard to set up. It’d be nice to avoid bikeshedding and get people immediate feedback on their PRs, esp for new contributors.
Although I would love to be in a place eventually where it becomes a CI check I imagine that may not be feasible in the short term. We may need to take baby steps to get there. Maybe like a bot that suggests code changes in a comment? It all depends what kind of appetite people have for a tool like this.
nixfmt development is dead since 2 years and it enforces very short lines with unecessary new lines at odd places. For example this creates very often a new line after url = because the url following it is to long. That doesn’t help though to make the code more readable.
I tried online demos of them few days ago. nixfmt is buggy, at least when making indentation in multi line strings, nixpkgs-fmt leaves the code almost untouched: I wanted them to format ugly code dumped by Expr::show(), and nixpkgs-fmt kept all the 1000+ character lines, excessive brackets, etc