From e82037f64a3aaf46670f7bcea0bbbe2000393fbc Mon Sep 17 00:00:00 2001
From: Arseny Balobanov <verytable@yandex.ru>
Date: Sun, 12 Mar 2023 19:06:41 +0300
Subject: [PATCH] [b/docs/gitfame_review_comments.md] Add chdir comment.

---
 docs/gitfame_review_comments.md | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

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
+```