Fork me on GitHub
#klipse
<
2017-02-18
>
moxaj17:02:29

if I remove (src-cb {:lang :clj :source ""}), it works, both in the repl, and with external libs

moxaj17:02:04

was that piece of code accidentally left there? ^^

Yehonathan Sharvit17:02:49

This is a huge mistake!

Yehonathan Sharvit17:02:59

Probably a wrong merge

Yehonathan Sharvit17:02:08

How did you find it?

moxaj17:02:31

I suspected the problem had something to do with load-ns

moxaj17:02:05

and eventually, I just stumbled upon that line

Yehonathan Sharvit17:02:25

such a stupid mistake 😞

Yehonathan Sharvit17:02:09

can u send me a pr and I’ll deploy it soon

moxaj18:02:16

could you approve my previous PR? it's been merged, but not approved

moxaj18:02:48

github still believes i'm 3 commits ahead

Yehonathan Sharvit18:02:08

I cannot find the button for approving the PR

moxaj18:02:36

it seems you've approved one of the commits. anyway, I've submitted the PR, it's only a slight noise

Yehonathan Sharvit18:02:35

In your PR I see also the contents of the previous PR

moxaj18:02:19

I suspect that's because of this approval addon? the previous 2 commits are already merged into master

moxaj18:02:59

oh wait, I think I know

moxaj18:02:15

I don't 😕

moxaj18:02:11

meh, i'll just create a fresh fork

moxaj18:02:36

nice, i'll test my lib right away when deployed

Yehonathan Sharvit18:02:29

For some reason in your code there was a hidden character at the beginning of io.cljs

Yehonathan Sharvit18:02:44

-<U+FEFF>(ns 
+(ns 

moxaj18:02:35

hm, thats a BOM, sorry about that

moxaj18:02:46

dunno how it got there

Yehonathan Sharvit18:02:26

I’m adding your klipse-test repo to my tests

Yehonathan Sharvit19:02:58

Please make sure you get klipse version 6.7.0

Yehonathan Sharvit20:02:53

Thanks to @moxaj, now klipse implicitly loads macro namespaces

Yehonathan Sharvit20:02:18

@arnaud_bos it should make your life easier with core.matrix