Ovviamente è una buona idea. Il commit di un cambio di semantica è abbastanza diverso dal commit di un cambio di indentazione.
Tuttavia, perché stai modificando il rientro? La regola generale della vecchia scuola era che il manutentore di un corpo di codice dovesse convivere con lo schema di indentazione dello sviluppatore originale. Ci sono delle eccezioni alle eccezioni a questa regola, ma l'intento della regola è di mantenere (1) evitare di modificare le guerre, e (2) evitare i commit che hanno un significato semantico zero.
Supponiamo di trovare un file nel codice di configurazione gestito che viola palesemente gli standard di codifica della tua organizzazione. È molto probabile che non sia l'unico file. Dovresti creare una nuova richiesta di modifica (o qualsiasi altra cosa venga chiamata nella tua organizzazione) per correggere tutti di tali problemi. Il commit potrebbe colpire qualsiasi cosa, da un file a migliaia, ma se la riformattazione viene eseguita da uno strumento ben testato, avrà un impatto semantico zero.
La nuova regola della scuola è che i commit / push saranno (a) automaticamente riformattati su uno stile comune concordato su commit / push, o (b) respinti se il codice in questione non si conformerà con quello stile. Ad ogni modo, non avrai problemi con il codice che si discosta dallo stile con cui tutti hanno accettato di rispettare.