Bob Nystrom

> As another data point, I did a search and found that ~35% of our organization's Dart projects (maintained by various independent teams) use a custom dartfmt line length of...

> don't force line lengths on us, keep it a setting. There are no plans to remove the option for specifying line length when running the formatter.

> It appears that Linus Torvalds agrees with us: > > https://www.theregister.com/2020/06/01/linux_5_7/ Linux uses 8 spaces of indentation, though, which Dart does not.

> Of course making the default to be overrideable is the bare minimum. It already is and has always been. Simply pass `--line-length` to your preferred length.

> maybe `analysis_options.yaml` is a good place for it? > it's got linter options, why not formatting options as well. The linter is built on top of the Dart analyzer....

Probably.

The [forthcoming](https://github.com/dart-lang/dart_style/issues/1403) [tall style](https://github.com/dart-lang/dart_style/issues/1253) handles argument lists and cascades differently. With the new style, you get: ```dart void main() { final object = outer( inner() ..innerProperty1 = 1 ..innerProperty2 =...

Yeah, I agree the current formatting is pretty bad. The new in-progress style gives you: ```dart abstract class StreamNotifierProviderBase< NotifierT extends AsyncNotifierBase, T > extends ProviderWithNotifier with FutureModifier { //...

I recently (#1511) fixed this in the [forthcoming](https://github.com/dart-lang/dart_style/issues/1403) [tall style](https://github.com/dart-lang/dart_style/issues/1253) formatter. With that change, you get: ```dart abstract class StreamNotifierProviderBase< NotifierT extends AsyncNotifierBase, T > extends ProviderWithNotifier with FutureModifier {...

I understand the desire here, but it goes against the fundamental goals of the formatter. Its goals, in order of descending priority, are: 1. Produce consistently formatted code. 2. End...