Remove ambiguity in params allignment

BackerKit / BackerKitProductFlow

This is where we keep track of how we do things.

1 stars 0 forks source link

Closed srt32 closed 7 years ago

srt32 commented 7 years ago

Why:

both the below options were previouls good but we've had a number of discussions around what we want. This change lands on a single one (pushing code as far left as possible).

This change addresses the need by:

removing the ambiguity and stating that when method params are broken onto multiple lines put one on each line and indent once from the method declaration. This change makes it easier to scan the list.

ryanclark2 commented 7 years ago

👍 An alternative edit would be to leave it as bad so there is even less ambiguity.

srt32 commented 7 years ago

Fantastic idea. Updated.