> On Tue, May 14, 2019 at 12:49:17PM -0700, Julius Werner wrote: > > One thing I'm missing is an option to maintain clang-format as an > > optional pre-submit hook that people can choose run over only the > > lines their patch is touching if they want to, like I suggested in > > that thread above. > Since everybody is free to do that without project coordination, we > don't need to ask for that to be project policy, no?
Well, we could still maintain a default .clang-format file and prepare a ready-to-install commit hook so that people don't have to figure out the details of clang-format-diff theirselves. I think it presents a valid third way of "how do we want to deal with auto-formatting". > I don't think there's anything bad enough to warrant calling off the > current round though, which would only create additional confusion. Yes, I didn't want to suggest that. I guess people who agree that formatting should be optional can vote against it here and then we could discuss details later if the option wins. _______________________________________________ coreboot mailing list -- [email protected] To unsubscribe send an email to [email protected]
