diff --git a/docs/gitfame_review_comments.md b/docs/gitfame_review_comments.md index 65ffa8d..399b1d1 100644 --- a/docs/gitfame_review_comments.md +++ b/docs/gitfame_review_comments.md @@ -182,3 +182,20 @@ var file []byte Нужно вкомпилить все зависимости в утилиту, например, с помощью embed. +## Работа с `os.Exec` + +Нельзя делать `os.Chdir`. + +После работы утилиты пользователь ожидает, что он останется в той же директории, в которй запускал утилиту. + +Неправильно: +``` +err := os.Chdir(repository) +cmd := exec.Command("git", "blame", "--porcelain", revision, "--", file) +``` + +Правильно: +``` +cmd := exec.Command("git", "blame", "--porcelain", revision, "--", file) +cmd.Dir = repository +```