valohai / django-allauth-2fa

Two-factor authentication for Django Allauth
Other
212 stars 50 forks source link

Disable pre-commit.ci autofixes #175

Closed akx closed 1 year ago

akx commented 1 year ago

The automagically pushed commits make non-squash merges impossible :/

valberg commented 1 year ago

I'd rather use squash merges (also produces a nicer git history) and get the benefit of pre-commit.ci autofix. But I can live without if you feel stronger about it @akx

akx commented 1 year ago

@valberg I prefer having the choice 😁 Sometimes well-built branch histories with a regular merge commit can be their worth in gold (e.g. when git bisecting or git blameing).