Feature melhorando npm scripts

angeliski commented 7 years ago

Lendo uma apresentação de npm-scripts percebi que haviam algumas mudanças que podiam agregar ao projeto. Eu tentei separar as execuções e adicionei as seguintes opções:

watch que permite iniciar observar os arquivos e executar o linter e os testes
coverage que permite executar o processo completo de cobertura. Isso será usado pelo Travis
eslint:fix que permite corrigir os arquivos conforme regras do linter
Adicionada a dependencia npm-run-all que permite execuções multiplas de forma organizada e paralela
O README foi atualizado, contendo mais informações sobre o desenvolvimento e oferecendo um pequeno feedback sobre as contribuições.

coveralls commented 7 years ago

Changes Unknown when pulling c19001d686bc6f34fb4d38b200d348da1db5aa71 on feature-melhorando-npm-scripts into on master.

MarcelLZ commented 7 years ago

@angeliski que tal usar o githook de prepush lá no scripts? E nessa config, adicionar o lint e os testes fazendo com que seja obrigatório rodar estes dois antes de um push ;)

angeliski commented 7 years ago

@MarcelLZ Eu fiquei tentado a fazer isso, mas eu achei meio intrusivo... Se o cara tiver na branch dele, em teoria ele pode dar um push, talvez ele esteja rascunhando uma ideia, ou com pressa pra ir pra casa pra terminar isso e fez um commit parcial. Como o Travis trava isso no PR, eu preferi não adicionar essa parada

MarcelLZ commented 7 years ago

Entendi, faz muito sentido :+1:

frontendbr / eventos-api

Feature melhorando npm scripts #46