Closed paulreeve closed 8 years ago
This looks good. Thanks for the PR.
Merging this PR I realise we don't really need a lot of the dependencies we have so I removed them here:
Awesome. Thanks @nabil-boag
@paulreeve I just realised that the README.md needs to be updated from your PR. It still mentions bower and bower_components.
ok i'll take a look.
Overview
Unlike most, I develop behind a corporate proxy which makes NPM just about bearable but Bower completely out of the question. Having inspected what libraries Bower was being used for, I realised NPM could fulfil all of these. This PR removes Bower from the equation completely which obviously helps me :smile: but also should simplify things slightly.
Details
app/build
instead ofapp/src
. I'm not sure if this was meant to run fromapp/src
but please let me know and I can put back.app/build
seems to make more sense.Tests?
All unit tests run and pass in Phantom and browsers: