|
|
|
|
|
|
by jakebailey
1321 days ago
|
|
|
No formatter would support having code indented like that at the top level for no reason, so that's not an option. Though, I have been looking into getting us to use a formatter, period (right now we don't). I didn't consider doing it step by step, no, but I'm not sure how I would really achieve that effectively. The reality is that the bulk of PRs are submitted by the team, and it's straightforward to get everyone to get their code in before this change, pause merges, rebase, and then continue as normal. I'd rather go for the "rip the band-aid off" method. |
|
|
That's what I was worried about, but I guess you can't have merge conflicts if you freeze the code.
Suppose you couldn't freeze the code, then what would you have done?