epacke / BigIPReport

Overview of your loadbalancer configuration
https://loadbalancing.se
8 stars 2 forks source link

json cleanup, bump version, tabs to spaces #123

Closed timriker closed 5 years ago

epacke commented 5 years ago

Hey Tim Sorry for the late reply. Great initiative! What do you think about keeping the indentation to 4xspace instead of 2? I believe we use that for the ps1 and javascripts.

timriker commented 5 years ago

The online view on github assumes tabs are 8, so I'd prefer to avoid any other assumption on tab size. However, If we are spaces, a smaller indent size works fine for me, no real preference.

epacke commented 5 years ago

Then, with you blessing, let's change the indentation to 4x space?

timriker commented 5 years ago

Fine with me. :) I'd recommend renaming "Move the content of this folder to the configured report root" to "underlay" if you would consider that.... :)