Skip to content

Can not build using bower [v3] #1832

Closed
naranjamecanica opened this Issue · 8 comments

3 participants

@naranjamecanica

I'm using bower to get and track the latest dev version. Because the builds in the bin directory are not always up to date I want to build it myself. Right now the gulp/tasks folder of is missing, the utils folder does exist. Is this intentional?

@englercj
Pixi.js member

Yes, the idea behind the bower/npm packages is that they are distributables. Bundles in bin if you want to use those directly, and source in src if you want to use browserify.

Really all you should get in that package is bin, src, and package.json. Anything else is something we should filter out. Thanks for reminded me to update that!

@naranjamecanica

Is it possible to have another bower file filter in the dev branch so if install that one, you can actually build it?

Or that a latest build is included, so it's more useful.

@englercj
Pixi.js member

Including tests scripts, build scripts and other repo data in a registry package is an antipattern. I really don't want to do that.

Instead can't you just clone the repo if it is for development? Does bower have something like "npm link" for these cases?

@drkibitz

npm can also install from git, or better yet ... a tarball. github itself provides every commit, tag, and branch as tarball which is a fast install via npm.

@drkibitz

@englercj thanks, totally agree with your "antipattern" comment there.

@drkibitz

@englercj while I'm commenting, I strongly suggest just using the "files": [] field in package.json rather than .npmignore (which IMO is an antipattern :wink:)

@drkibitz

Things to include:

  • bin
  • src
  • README.md
  • LICENSE
  • package.json

A couple of those may be included by default, can't remember which ATM.

@englercj
Pixi.js member

while I'm commenting, I strongly suggest just using the "files": [] field in package.json

Yeah for some reason in my head I had updated it, but this ticket reminded me that I didn't :P

@englercj englercj closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.