cljsjs

martinklepsch 2016-11-30T04:37:17.001393Z

@mikebelanger http://www.tonejs.org did not work for me but http://tonejs.org did — maybe that www isn't set up to work?

mikebelanger 2016-11-30T13:28:11.001397Z

@martinklepsch oh neat, I didn't realize that. I made a PR where the url was changed to point directly to github repo, but http://tonejs.org is much easier -- plus they're probably going to make a non-github homepage eventually for that.

mikebelanger 2016-11-30T14:29:38.001398Z

@martinklepsch just updated PR to include http://tonejs.org url

juhoteperi 2016-11-30T14:38:07.001399Z

@mikebelanger There is a conflict in the PR, the version number in readme is already fixed (they are automatically updated)

mikebelanger 2016-11-30T14:41:15.001400Z

@juhoteperi oh ok, what should I do, just delete my PR?

juhoteperi 2016-11-30T14:41:27.001401Z

Other changes are still fine

juhoteperi 2016-11-30T14:44:43.001404Z

I fixed the conflict

mikebelanger 2016-11-30T14:46:35.001405Z

@juhoteperi ah cool, thanks. If that happens again, should I just merge like you just did?

juhoteperi 2016-11-30T14:54:46.001406Z

Merge is okay

juhoteperi 2016-11-30T14:54:59.001407Z

Rebase is even better

juhoteperi 2016-11-30T14:55:44.001408Z

Or just commit --amend & force push in case of a single commit

mikebelanger 2016-11-30T16:10:34.001409Z

ok, thanks I'll have to remember that for next time. It doesn't look like the site link works, does it just take time to reflect the changes http://cljsjs.github.io?

martinklepsch 2016-11-30T16:16:30.001410Z

@mikebelanger might take a few mins, think the site is updated using a cron job

mikebelanger 2016-11-30T17:26:45.001412Z

Do you guys know of any cljsjs packages that use multiple levels of Google Closure compilation? ie) advanced, whitespace, simple?

mikebelanger 2016-11-30T17:30:20.001413Z

oh, found one, please disregard last question

mikebelanger 2016-11-30T20:48:20.001414Z

If you have multiple .ext.js files for your project, and they're all in the common subdir in the fileset, will Google Closure incorporate all of them during its compilation (provided its advanced?)

juhoteperi 2016-11-30T20:51:16.001415Z

Closure doesn't care about extension or path

juhoteperi 2016-11-30T20:51:47.001416Z

ClojureScript compiler reads the deps.cljs files and builds options for Closure from those

juhoteperi 2016-11-30T20:52:11.001417Z

Cljsjs tooling usually creates the deps.cljs file

mikebelanger 2016-11-30T20:59:45.001420Z

@juhoteperi ah I see thanks

mikebelanger 2016-11-30T22:12:10.001421Z

So in cljsjs packaging instructions, it asks users to compile things with the :advanced flag. It looks like my projects work on anything *but* :advanced, and I suspect it's something to do with my externs. But it works fine on whitespace and simple, is it still acceptable to be on cljsjs?