search

voxpupuli / puppet-lint-strict_indent-check

indent check for puppet-lint
Mozilla Public License 2.0
4 stars 11 forks source link

Make expected indentation configurable #1

Closed madAndroid closed 9 years ago

madAndroid commented 9 years ago

Would it be possible to make the indentation level configurable? ie 4 spaces vs 2 spaces - not all organisations follow the style guide to the letter, and for us having the indentation set to 2 spaces is not viable.

relud commented 9 years ago

Sure, I'll see if I can get around to it next week (today and tomorrow are holidays in the U.S.) On Nov 27, 2014 6:07 AM, "madAndroid" notifications@github.com wrote:

Would it be possible to make the indentation level configurable? ie 4 spaces vs 2 spaces - not all organisations follow the style guide to the letter, and for us having the indentation set to 2 spaces is not viable.

— Reply to this email directly or view it on GitHub https://github.com/relud/puppet-lint-indent-check/issues/1.

madAndroid commented 9 years ago

That's awesome - thanks for considering that :) I'd do it myself and submit a PR, but I'm not familiar with puppet-lint, and can't see where to even begin

On Thu Nov 27 2014 at 8:14:23 PM Daniel Thornton notifications@github.com wrote:

Sure, I'll see if I can get around to it next week (today and tomorrow are holidays in the U.S.) On Nov 27, 2014 6:07 AM, "madAndroid" notifications@github.com wrote:

Would it be possible to make the indentation level configurable? ie 4 spaces vs 2 spaces - not all organisations follow the style guide to the letter, and for us having the indentation set to 2 spaces is not viable.

— Reply to this email directly or view it on GitHub https://github.com/relud/puppet-lint-indent-check/issues/1.

— Reply to this email directly or view it on GitHub https://github.com/relud/puppet-lint-indent-check/issues/1#issuecomment-64819386 .

relud commented 9 years ago

fixed in 1.1.0

madAndroid commented 9 years ago

Brilliant, thanks a lot!