Improve gulpfile - Githubissues

xpostudio4 / metacommunity

New website of the meta community

3 stars 4 forks source link

Improve gulpfile #18

Closed francisbrito closed 8 years ago

francisbrito commented 8 years ago

Fix linting errors.
(Partially) Fix browser tab being launched before server started.

I'd like to know your opinions regarding this change. @viviangb @xpostudio4 @eatskolnikov

eatskolnikov commented 8 years ago

@francisbrito Haz squash http://stackoverflow.com/questions/5189560/squash-my-last-x-commits-together-using-git, si no sabes cómo yo te enseño.

francisbrito commented 8 years ago

@eatskolnikov Agrupar los cambios hechos por varios commits en uno solo tiene alguna ventaja? O se trata de alguna convención que sigue este proyecto?

eatskolnikov commented 8 years ago

@francisbrito se ven mas kawaii los commits

xpostudio4 commented 8 years ago

@francisbrito veo multiples commits que ya enviaste en este, es posible que solo incluyas los relacionados con el gulp file, sino el commit message no hace sentido. Creo que los commits que deberías dejar son el 6de34e2 , ddce1f6 y el d8c3ac0.

Algo que no veo es la documentación de que se puede usar gulp para comenzar el server, sería bueno para informarles a los usuarios.

francisbrito commented 8 years ago

@xpostudio4 los otros merges que aparecen ahí son a cambios que ya fueron aplicados al branch master de este repositorio. Es decir, no producirán cambios nuevos al proyecto. Dicho eso, si es necesario remover procederé a averiguar como hacerlo sin tener que revertir esos cambios (Si conocen alguna manera @eatskolnikov, @xpostudio4, por favor haganmelo saber).