[b/docs/gitfame_review_comments.md] Add note on style.
This commit is contained in:
parent
e82037f64a
commit
6959d8ac5d
1 changed files with 22 additions and 2 deletions
|
@ -189,13 +189,33 @@ var file []byte
|
|||
После работы утилиты пользователь ожидает, что он останется в той же директории, в которй запускал утилиту.
|
||||
|
||||
Неправильно:
|
||||
```
|
||||
```golang
|
||||
err := os.Chdir(repository)
|
||||
cmd := exec.Command("git", "blame", "--porcelain", revision, "--", file)
|
||||
```
|
||||
|
||||
Правильно:
|
||||
```
|
||||
```golang
|
||||
cmd := exec.Command("git", "blame", "--porcelain", revision, "--", file)
|
||||
cmd.Dir = repository
|
||||
```
|
||||
|
||||
## Стиль
|
||||
|
||||
### NIT Используйте общие var и const декларации
|
||||
|
||||
Для однородных переменных и констант нет необходимости писать `var` перед каждой строкой
|
||||
|
||||
Вместо
|
||||
```
|
||||
var flagRepo = flag.String("repository", ".", "repo")
|
||||
var flagRev = flag.String("revision", "HEAD", "revision")
|
||||
```
|
||||
|
||||
Можно написать
|
||||
```golang
|
||||
var (
|
||||
flagRepo = flag.String("repository", ".", "repo")
|
||||
flagRev = flag.String("revision", "HEAD", "revision")
|
||||
)
|
||||
```
|
||||
|
|
Loading…
Reference in a new issue