search

Iltotore / iron

Strong type constraints for Scala
https://iltotore.github.io/iron/docs/
Apache License 2.0
427 stars 37 forks source link

Use fewerBraces in Iron codebase #210

Closed Iltotore closed 2 months ago

Iltotore commented 5 months ago

Iron uses Scala 3's identation-based control syntax. Now that it uses Scala 3.3.1, we can use fewer braces.

It should be easy to migrate the syntax:

zaxxel commented 2 months ago

Hello! Tell me, please, is it necessary to format all existing code? I'm asking because with the current configuration and version of ScalaFMT, the formatting check returns 58 misformatted files. If it is updated to 3.8.1, then 12 more will be added.

Iltotore commented 2 months ago

Formatting the existing code of Iron is not necessary but I believe it is welcome and I don't see any cons to do it: this will not break any kind of compatibility and the major part of files to reformat are unit tests which use Scala 3.1's "partial few braces" syntax.