cljsjs

lwhorton 2016-09-16T14:10:00.000621Z

@juhoteperi I noticed youre the official/unofficial contributor for cljsjs/highstock. Is there a reason the latest 4.2.2 has a :requires [“cljsjs.jquery”]?

lwhorton 2016-09-16T14:10:39.000622Z

As best I can tell since post-4.1 they no longer have jquery as a dep

juhoteperi 2016-09-16T14:10:41.000623Z

No idea, I just merge PRs

juhoteperi 2016-09-16T14:11:16.000624Z

https://github.com/cljsjs/packages/pull/399/files

lwhorton 2016-09-16T14:11:20.000626Z

I see. My mistake. @jimlar ^ perhaps was the original contributer?

adamfrey 2016-09-16T20:09:38.000635Z

I accidentally pushed my semantic-ui update commit to master, instead of a new branch with PR, sorry. Probably isn’t a big deal

adamfrey 2016-09-16T20:09:55.000636Z

here’s the commit: https://github.com/cljsjs/packages/commit/9225e43b1677aa1192dcdb2f93428e7da46ba620

juhoteperi 2016-09-16T20:10:07.000638Z

Yeah, it's okay