canonical / canonical/cloud-utils

[LP: #1836601] whitespace cleanups needed (tabs to spaces)

Open
#42 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Shell
Stars
124
Forks
36
PR merge metrics
No merged PRs in 30d

Description

Originally reported by: [Scott Moser](https://launchpad.net/~smoser) (@smoser) on 2019-07-15
Original bug URL: https://bugs.launchpad.net/cloud-utils/+bug/1836601
Last Launchpad Status: Triaged
Last Launchpad Severity: Wishlist

---

When I started cloud-utils long ago, I was under the misguided view that tabs could be used reliably in source code. I have now accepted that without a machine formatted (go fmt) that is not the case.

An example can be seen in cloud-utils itself that has mixed tabs and spaces for indentation. smoser fail.

So, this bug is a request to:

a.) change tabs used for indentation to 4 spaces.
b.) change the vi modeline to be correct, and make sure it is present at the end of all files.

The correct modeline for 4 spaces indentation is:
```
# vi: ts=4 expandtab
```

Note... if someone insists that the right fix is to remove modelines entirely [1] then I would accept that, but *something* needs to insist on consistent indentation, and I used vi modelines as a mechanism for declaring the style in a file.

--
[1] https://security.stackexchange.com/questions/36001/vim-modeline-vulnerabilities

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.