Update mechanism to choose indentation style

DTW-DanWard / PowerShell-Beautifier

A whitespace reformatter and code cleaner for Windows PowerShell and PowerShell Core

MIT License

334 stars 38 forks source link

Update mechanism to choose indentation style #22

Closed bnygld closed 7 years ago

bnygld commented 7 years ago

Hi Dan!

First of all, thanks for this module, I've got a passion for well-formatted code and PSScriptAnalyzer is sorely missing an auto-correct feature a la rubocop.

I've made quite a drastic change to the indentation param - a choice of the major indentation options (without prejudice).

Let me know if this is something you'd consider accepting or if there are any changes you want

cheers Ben

DTW-DanWard commented 7 years ago

Hey Ben -

Taking a quick look... looks perfect! Yep, I agree with you, leaving it open to any type of index text is probably silly; there really are only three options we use. Having an explicit choice is also a better user interface; with text someone might accidentally type 3 spaces " " instead of 4 but now that problem goes away.

Thank you for updating all the various locations - including the test script and docs! I will merge later today.

-Dan