New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Inconsistent indentation levels #32

Closed

seppestas opened this issue Jan 29, 2016 · 1 comment

seppestas commented Jan 29, 2016

There seems to be some inconsistency in the indentation levels used by this SDK.

Most files seem to use an indentation level of 2 spaces, however some files, e.g thing/index.js and example/thing-example.js use 3 spaces.

Even more OCD triggering examples are device/index.js and examples/lib/cmdline.js, which seem to use a mix of 2, 3 and 4 spaces indentation.

seppestas changed the title ~~Inconsistent indentation~~ Inconsistent indentation levels

seppestas added a commit to seppestas/aws-iot-device-sdk-js that referenced this issue


          Normalized indentation to 2 spaces, fixes aws#32

550311e

All files now consistently use the same indentation.
This was mostly done using [js-beatify](https://github.com/beautify-web/js-beautify)

Contributor

gkwicker commented Jan 31, 2016

Hi Seppe, thanks for letting us know about the indentation level inconsistency. We'll look into fixing this in an upcoming release.

gkwicker closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment