search

mozillascience / software-discovery-dashboard

MIT License
19 stars 8 forks source link

Refactor the Dashboards dependencies #47

Closed lukecoy closed 8 years ago

lukecoy commented 8 years ago

Refactor ALL the dependencies.

Gulp was a pain for us and was turning into a giant anti pattern, that was becoming (for us) a little too unmaintainable. We were kind of entering a dependency hell that I wasn't pleased with. So we remove gulp in favor of NPM.

On top of gulp getting chopped, the express boilerplate also got removed, as it also introduced a lot of YAGNI dependencies that we don't care about (at least, not yet). I replaced the express boilerplate with more readable code and a more sensical file structure.

@Lettuceman44 @mok4ry

lukecoy commented 8 years ago

@Lettuceman44 after this gets merged, we'll need to kill jenkins (thank god) and start using travis. that'll free up some server processing power and integrates directly into github (as in, you can see builds kicking off right here on the PR when we hook it up.. super cool).

lukecoy commented 8 years ago

-290 lines. amazing

varzaman commented 8 years ago

Looks good!