Segundo a documentação de communicate
o retorno é uma tupla contendo a saída padrão e a saída de erros. Além disso, quando não está no modo texto, o retorno é um objeto do tipo bytes (dá para você ver pelo b
no início da saída). Além disso, como o comando não deu nenhum erro, o segundo elemento da tupla é None
.
Sendo assim, você precisa pegar o primeiro elemento da tupla (que é a saída padrão do comando) e transformá-lo em string, usando o método decode
.
Além disso, o comando está retornando os campos separados por um TAB (\t
), então basta usar replace
para substituí-lo por vírgula:
data = subprocess.Popen(["git", "-C", "github_repos/" + token + "/" + token, "shortlog", "-s", "-n"], stdout=subprocess.PIPE).communicate()
insights.write(data[0].decode().replace('\t', ', '))
Outra opção é ler o stdout
do processo linha a linha:
import subprocess
# atenção, aqui não chama communicate()
data = subprocess.Popen(["git", "-C", "github_repos/" + token + "/" + token, "shortlog", "-s", "-n"], stdout=subprocess.PIPE)
for s in data.stdout:
insights.write(s.decode().replace('\t', ', '))
Outra opção é setar o argumento universal_newlines
, assim o retorno serão strings em vez de bytes, eliminando a necessidade de chamar decode
:
data = subprocess.Popen(["git", "-C", "github_repos/" + token + "/" + token, "shortlog", "-s", "-n"], stdout=subprocess.PIPE, universal_newlines=True).communicate()
insights.write(data[0].replace('\t', ', '))