Fork me on GitHub
#cljsjs
<
2016-11-30
>
martinklepsch04:11:17

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

mikebelanger13:11:11

@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.

juhoteperi14:11:07

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

mikebelanger14:11:15

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

juhoteperi14:11:27

Other changes are still fine

juhoteperi14:11:43

I fixed the conflict

mikebelanger14:11:35

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

juhoteperi14:11:46

Merge is okay

juhoteperi14:11:59

Rebase is even better

juhoteperi14:11:44

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

mikebelanger16:11:34

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?

martinklepsch16:11:30

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

mikebelanger17:11:45

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

mikebelanger17:11:20

oh, found one, please disregard last question

mikebelanger20:11:20

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?)

juhoteperi20:11:16

Closure doesn't care about extension or path

juhoteperi20:11:47

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

juhoteperi20:11:11

Cljsjs tooling usually creates the deps.cljs file

mikebelanger22:11:10

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?