Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add gulp and bower to the list of devDependencies #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

add gulp and bower to the list of devDependencies #9

wants to merge 1 commit into from

Conversation

davemo
Copy link

@davemo davemo commented Sep 17, 2015

  • this allows users to avoid having globally installed versions of these
    modules so that the only thing they need to do to contribute is:

npm i
npm test

- this allows users to avoid having globally installed versions of these
modules so that the only thing they need to do to contribute is:

npm i
npm test
@chemerisuk
Copy link
Owner

@davemo I put all dependencies into separate project https://github.com/chemerisuk/better-dom-boilerplate. Usage of bower is deprecated - the community decided to use NPM for everywhere. And gulp is awesome, you do need to have it to be installed globally :)

@jasonkarns
Copy link

I'm 👍 on this. This package requires gulp for any of the npm scripts to function and global installs are an antipattern.

According to the karma config in package.json, this module is also expecting bower to be used as well:

"bower_components/better-dom/dist/better-dom.js",
"bower_components/better-i18n-plugin/dist/better-i18n-plugin.js",
"bower_components/better-popover-plugin/dist/better-popover-plugin.js"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants