Fork me on GitHub
#cljsjs
<
2018-11-21
>
miikka06:11:01

@jsa-aerial If you have a patch ready to update vega-embed, please submit a PR and let's merge it.

jsa-aerial15:11:10

OK, done and checks pass. But I see that you went ahead and merged the 3.23.1 - I used my fork as updated to latest CLJSJS before that update, so there are 'conflicts' because the version 3.24.1 is being compared to 3.16.1-1 - same with old vega and vega-lite in build.boot.

jsa-aerial15:11:07

OK, in order to prevent this sort of nonsense in the future, I would be willing to be the point person to keep these (the various vega libs) properly updated. I think I saw a place where one can sign up for that but can't find it now. Can you point me to that?

miikka16:11:11

I don't think that I merged any changes to vega-embed? The boot-cljsjs dependency has been updated and that's why there are the conflicts.

miikka16:11:45

I resolved the conflicts now

jsa-aerial16:11:40

Yes, that looks like what I meant. So, do I just add stuff to that and give PR for it?

jsa-aerial16:11:59

OK, sorry, on conflicts it looked like it was complaining about something else - I must have misunderstood. THANKS for the update!