cljdoc

https://cljdoc.org/ & https://github.com/cljdoc/cljdoc
martinklepsch 2018-11-16T14:01:40.090300Z

Ok, responsiveness stuff is live https://cljdoc.org πŸ“±

4πŸ‘2😍
2018-11-16T15:08:07.090900Z

Haven’t found the easter egg yet hahah

martinklepsch 2018-11-16T15:16:56.091300Z

I can tell you if you want πŸ˜„

2018-11-16T15:20:35.091500Z

No don't spoil it yet!

2018-11-16T15:45:41.091800Z

Found it! I’ll tell my friends πŸ˜„

martinklepsch 2018-11-16T15:51:56.092Z

πŸ˜„ πŸ˜„

martinklepsch 2018-11-16T15:52:19.092200Z

I'd like to add a few more fun messages (maybe with different colors even)

martinklepsch 2018-11-16T15:53:01.092400Z

so if you have ideas, go for it πŸ˜„

2018-11-16T15:54:36.092600Z

That'd be fun! I'll sneak it in in my next PR 😜

2018-11-16T16:47:15.094900Z

Currently, when we try to use the main page of cljdoc to find the documentation of the re-frame project, it is hard to get to the right auto-completion. I got it by typing day8.re-frame which auto-completes to re-frame in the list, but typing re-frame alone does not show the project in the list.

martinklepsch 2018-11-16T18:08:20.095700Z

@vincent.cantin The search is backed by Clojars' search which unfortunately is... pretty bad.. πŸ˜„

martinklepsch 2018-11-16T18:10:04.096600Z

We might want to provide our own search backed by our own DB but that hasn't been a very high priority so far

martinklepsch 2018-11-16T18:10:26.097200Z

Although it probably should be since the search is one of the first things users interact with πŸ™ˆ

1βž•
martinklepsch 2018-11-16T18:10:40.097500Z

Is anyone actively working on this? https://github.com/cljdoc/cljdoc/issues/203

elarouss 2018-11-17T10:44:07.111800Z

i've just made a pull request for it, a one liner hhhh

martinklepsch 2018-11-17T10:54:33.112Z

@ichigo haha, you think the pull request wasn't worth it? πŸ˜„

martinklepsch 2018-11-17T10:55:14.112200Z

Change looks good, did you test it locally?

elarouss 2018-11-17T22:20:28.127700Z

@martinklepsch it's a beginner issue after all, and you gave me all the pieces hhhh

elarouss 2018-11-17T22:22:55.127900Z

i tested that within the repl

martinklepsch 2018-11-17T22:23:25.128700Z

I deployed it and it works great! πŸ‘

elarouss 2018-11-17T22:25:29.129Z

great, i want to try another issue, but i guess i need a day or 2 to read about the libraries you are using

elarouss 2018-11-17T22:25:59.129200Z

i didn't even know Integrant exists besides component and mount

martinklepsch 2018-11-17T22:26:21.129800Z

Hehe, yeah. Let me know if you have any questions

elarouss 2018-11-17T22:27:16.130Z

i have just questions now hhhh so i doing some investigation first is reasonable

elarouss 2018-11-17T22:27:33.130400Z

thank you again for being welcoming and helpful

martinklepsch 2018-11-17T22:28:45.131500Z

Here’s another issue that might be a good starter and requires changes in a few more places https://github.com/cljdoc/cljdoc/issues/199

elarouss 2018-11-17T22:38:06.131900Z

i see, i'll try fixing it πŸ™‚

martinklepsch 2018-11-17T22:55:46.132900Z

Let me know if you run into problems :) and send me your address in a DM :)

jsa-aerial 2018-11-16T20:44:08.100Z

So, when I try a build for a repo the job fails with an error that makes no sense. The job is Jobsmartinklepschcljdoc-builder master 7138 if that helps. The error is Unable to resolve var: coll? . coll? is just part of clojure.core.

2018-11-16T20:50:36.100800Z

Would you paste the full stack trace here? Thanks

jsa-aerial 2018-11-16T20:54:37.101Z

It's gigantic, but here it is:

jsa-aerial 2018-11-16T20:55:49.101200Z

Doesn't work - slack says text is too long. Maybe a partial will help:

jsa-aerial 2018-11-16T20:58:07.101400Z

Nope too long. Let's try a partial in a file:

jsa-aerial 2018-11-16T20:59:12.101600Z

Nope, need to upgrade to paid plan for that.

jsa-aerial 2018-11-16T20:59:41.101800Z

stderr -------------------------------------------------------------------- Exception in thread "main" clojure.lang.ExceptionInfo: Could not analyze aerial/hanami/common.cljc {} at clojure.core$ex_info.invokeStatic(core.clj:4739) at clojure.core$ex_info.invoke(core.clj:4739) at cljdoc.analysis.impl$codox_config$ex_handler__983.invoke(impl.clj:21) at codox.reader.clojurescript$read_file.invokeStatic(clojurescript.clj:95)

jsa-aerial 2018-11-16T21:00:00.102Z

at clojure.main.main(main.java:37) Caused by: clojure.lang.ExceptionInfo: Unable to resolve var: coll? in this context at line 1446 file:/home/circleci/.m2/repository/com/rpl/specter/1.1.1/specter-1.1.1.jar!/com/rpl/specter.cljc {:file "file:/home/circleci/.m2/repository/com/rpl/specter/1.1.1/specter-1.1.1.jar!/com/rpl/specter.cljc", :line 1446, :column 3, :tag :cljs/analysis-error} at clojure.core$ex_info.invokeStatic(core.clj:4739) at clojure.core$ex_info.invoke(core.clj:4739) at cljs.analyzer$error.invokeStatic(analyzer.cljc:706)

jsa-aerial 2018-11-16T21:00:25.102200Z

Two small snips that might give you enough

martinklepsch 2018-11-16T21:32:20.102900Z

Since that’s on CircleCI could you provide the URL to the build?

elarouss 2018-11-16T21:42:01.103100Z

yeah thanks, i've seen that issue, so if i understand it right, the "more recent version" notification should not appear in case of the newer releases are just Snapshots, right ?

elarouss 2018-11-16T21:44:46.103300Z

i think i need to know the shape of the cache bundle map passed to the cljdoc.bundle.more-recent-version function

jsa-aerial 2018-11-16T22:02:28.103500Z

https://circleci.com/gh/martinklepsch/cljdoc-builder/7138

jsa-aerial 2018-11-16T22:02:37.103800Z

I think that is what mean

martinklepsch 2018-11-16T22:08:04.104Z

@ichigo here's how :latest gets into the cache bundle: https://github.com/cljdoc/cljdoc/pull/168/files#diff-a9925a94dd89b08dad3afbe0a721dc30R118

martinklepsch 2018-11-16T22:08:17.104200Z

In general that diff will be useful for a bit more context

martinklepsch 2018-11-16T22:09:50.104400Z

That's a weird looking issue, I'll take a closer look tomorrow, thanks for bringing it up!

jsa-aerial 2018-11-16T22:15:11.104600Z

βœ”οΈ

elarouss 2018-11-16T22:19:55.104800Z

@martinklepsch i'm sorry for being so beginner here 😳 , so we need to not put the snapshots in the db at first place i guess

elarouss 2018-11-16T22:21:14.105100Z

but that looks out of the scope of that issue

martinklepsch 2018-11-16T22:22:07.106700Z

No worries 😊 I think it might be easiest to just filter the list of versions to remove any versions ending with -SNAPSHOT before taking the last version in the line I linked to.

martinklepsch 2018-11-16T22:23:04.107400Z

Hope that makes sense.

martinklepsch 2018-11-16T22:23:43.108700Z

It might be helpful to just get the project running and experiment with the get-documented-versions function at the REPL

martinklepsch 2018-11-16T22:24:34.110300Z

If you’re having trouble setting up a REPL with your editor (which is very handy) I can help you tomorrow or you can ask for help in #editors or #beginners

martinklepsch 2018-11-16T22:24:49.110900Z

Good luck! πŸ˜„ (and good Night for now :)

elarouss 2018-11-16T22:25:34.111100Z

hhhh thank you, i'm running the repl now, but yeah i need to use it from inside emacs

elarouss 2018-11-16T22:26:39.111300Z

yeah that the first thing i thought about the -SNAPSHOT part, but because i'm like a stranger in a new town hhhh i'm too doubtful

elarouss 2018-11-16T22:27:35.111500Z

thank you, good night for you too :simple_smile: