Skip to content

Conversation

thiagodds
Copy link
Contributor

Pessoa, reescrevi o log usando logging como foi sugerido.

Essa é a primeira implementação, uma sugestão. Por favor façam as devidas críticas e sugiram melhorias.

É a primeira vez que estou usando loggging assim como é a primeira vez que tento participar de um projeto open. Conto com a paciência para me ajudarem e estou disposto a melhorar o que for preciso.

O que mais for preciso para ajudar no projeto podem contar comigo e entrar em contato comigo. Na medida do possível (tempo livre) eu posso ajudar.

@aleborba
Copy link
Member

Ae @iurisilvio, como a issue é sua, nada mais justo que você analisar e, se pá, aceitar :)

@iurisilvio
Copy link
Contributor

Bem melhor assim! Agora só vou te pedir pra corrigir os detalhes.

Acho que você deu um pip freeze para listar as dependências, o que adicionou várias dependências. Você só adicionou o PyYAML, é melhor só adicionar ela manualmente no requirements (PyYAML==3.10). O freeze lista tudo que está instalado no seu ambiente.

Depois disso, faça um squash do seu pull request. Ele vai transformar seu pull request em um único commit só com o que foi realmente alterado. http://blog.steveklabnik.com/posts/2012-11-08-how-to-squash-commits-in-a-github-pull-request

Uma dica para suas próximas contribuições (não atrapalha em nada esse pull request), evite trabalhar direto no seu master. Trabalhe sempre em uma branch (git checkout -b sua_branch), isso simplifica o seu processo de manter o seu repositório atualizado com o upstream.

Obrigado pelo pull request! Continue contribuindo, não só com o Postmon! Qualquer dúvida, só perguntar.

Adequando arquivos a PEP8.

Correções PEP8.

Log usando o módulo logging.

Configuração do arquivo yaml via string.

Correção no requirements.txt.
@thiagodds
Copy link
Contributor Author

Feito!

Obrigado pelas dicas.

iurisilvio added a commit that referenced this pull request Jan 12, 2014
Logs no CEP tracker. Close #63.
@iurisilvio iurisilvio merged commit 6176012 into PostmonAPI:master Jan 12, 2014
@iurisilvio
Copy link
Contributor

👍

Pronto, seu primeiro pull request está integrado. Obrigado!

A hora que alguém decidir usar o logging em outras partes, é só refatorar seu código, será bem simples.

@aleborba
Copy link
Member

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants