search

flo-schu / diverssite

0 stars 1 forks source link

Autoformat HTML Templates for Sidebar and Eventslist #66

Closed hf-kklein closed 2 years ago

hf-kklein commented 2 years ago

using [STRG]+[L] in PyCharm

hf-kklein commented 2 years ago

The thing is: you shouldn't mix formatting and feature PRs. https://mtlynch.io/code-review-love/#6-separate-functional-and-non-functional-changes

flo-schu commented 2 years ago

The thing is: you shouldn't mix formatting and feature PRs. https://mtlynch.io/code-review-love/#6-separate-functional-and-non-functional-changes

I can see the problem, but then, wouldn't it be better to do reformatting after the feature PR is closed?