taxweb/ruby_danfe — pull request #5 by cesarjr (2013-12-28 17:06:49+00:00)
Title: Install bundler and improve the basic gem structure.

Olá pessoal! Tudo bem? Eu espero que sim.

Em primeiro lugar eu gostaria de parabenizá-los pelo projeto. Obrigado pela iniciativa de abrir este código. Ele poupará um bom tempo de nosso time na confecção do DANFE.

Em segundo lugar em gostaria de pedir desculpas, caso eu tenha mexido em coisas demais. É que eu achei o projeto tão bacana que eu me empolguei. Eu gostaria de fazer mais modificações no fonte, mas primeiro vou submeter estas pra ver se o que eu penso sobre o projeto bate com o que vocês esperam dele.

Minha segunda ação seria começar uma cobertura mínima de testes e depois separar um pouco o conteúdo do arquivo ruby_danfe.rb. Acho que poderíamos quebrar em arquivos menores. Vou aguardar pra ver se este me pull request aprova.

Estou torcendo pela aprovação, pois estou muito motivado para continuar.

Então tá... segue abaixo o que basicamente este pull request faz:

* Ajustado o gemspec.

* Adicionado o gemfile para facilitar a produção. O gemfile.lock foi colocado no gitignore, pois em gems não é uma boa versioná-lo.

* Pequenas melhorias nas descrições do readme.

* O projeto estava sem nenhum tipo de licença especificado. Tomei a liberdade de colocar a do MIT. Isso foi feito tanto no gemspec quanto no readme.

* Coloquei ainda no readme também um sessão convidando para quem quiser contribuir.

* Removi do git arquivos ruby_danfe*.gem (gems compiladas) que me pareceram desnecessários. Coloquei isso também no .gitignore. Normalmente isso só serve pra fazer o envio pro rubygems e pra instalação local.

* Removi o build.sh, pois rake install faz a mesma coisa.

* Percebi também que não está sendo gerado tags para cada versão publicada. Se foi usado o rake release ele fará isso por você.

* Muitas modificações eu me baseei aqui: http://asciicasts.com/episodes/245-new-gem-with-bundler. Já usei isso em outras gems e ficou bem bacana. Acho que agora a gem tá dentro do usual da comunidade (não que estivesse errado, porém, estava fora do padrão).

* Eu removi também o arquivo translate.rb. Ele não tinha nenhuma documentação de uso no README e me parece que foi algo usado temporariamente. Pareceu não ter serventia. Pode ser que eu tenha cometido um erro removendo-o, mas não vi diferença no resultado final dos arquivos pdfs.

* Criei a tareka rake console para carregar um irb com todas as classes do projeto pré-carregas. Coloquei também um exemplo no README de como gerar o PDF a partir de uma string de XML.

Sugestões para o responsável pelo projeto:

* Decidir se a MIT é uma boa licença (ou usar outra, mas não deixar sem licença, pois isso pode dar rolo.)

* Gerar tags para cada versão. Lembrando que o rake release ajuda nesta tarefa.