Building and testing about bacon.js HOT 17 CLOSED

Oh and where to put the generated javascripts. So far it's worked ok to host them on github too. At least I like them to be easily available and up to date. Any other ideas for how to generate/host these files?

from bacon.js.

So we still can remove the assert using grunt. We can define a custom tasks for this. We coud also run the test with grunt, but I'm not sure if this is necessary. Using npm test is also the command that travis uses to run the test by default.

For generated files, whats wrong on hosting them from github. Also the homepage could just gh-pages branch. And a grunt task that generates the files from the markdown documents.

from bacon.js.

I guess it's best to keep the generated files on github in the future too. However I find it a bit odd to put generated files to lib/ with jquery, maybe dist/ would be more intuitive dunno. Anyways it's a minor thing.

from bacon.js.

👍 for dist/, lib/ only for jquery. I work on removing the assertion with grunt, so we can skip the cake file, will change this as well.

from bacon.js.

Btw it seems that the world is not ready yet for grunt 0.4.x. Contrib plugins seem to work out-of-the-box but e.g. grunt-shell or grunt-exec fail. They work like a charm with latest 0.3.x. Should consider switching to 0.3.x unless we want to live on the bleeding edge.

from bacon.js.

+1 for ditching Cakefile in favor of grunt.

from bacon.js.

Made a pull request #65

from bacon.js.

Merged. Now there's still something amiss? I did

npm install
./grunt

And the result is

>> Local Npm module "grunt-contrib-copy" not found. Is it installed?
Warning: Task "copy" not found. Use --force to continue.

from bacon.js.

Same for me. It's because grunt-contrib-copy has no specific version, instead it references current master (https://github.com/gruntjs/grunt-contrib-copy/archive/master.tar.gz). npm-shrinkwrap ignores the dependency for this reason (?). If you rm npm-shrinkwrap.json && npm install && ./grunt should work fine. However, I hope there is a way of specifying the version for grunt-contrib-copy so that we could continue using npm-shrinkwrap. I guess this would just work with grunt 0.3.x..?

from bacon.js.

To clarify: npm install ignores package.json if npm-shrinkwrap.json is present.

from bacon.js.

So I think the best should be to go back to grunt 0.3.

from bacon.js.

Sounds fair.

from bacon.js.

Ok if I do this?

from bacon.js.

I've just started.

from bacon.js.

👍 go for it.

from bacon.js.

#66 Merged. Now it seem to work fine.

I put /dist under version control for now, as there's no other distribution mechanism for the pre-built javascripts.

I updated references in [readme,package.json,component.json] to point to dist instead of lib.

I published 0.0.12

from bacon.js.

Now that readme is updated, this can be closed. Thanks guys!

from bacon.js.