Today I put up this PR, which includes the code change needed to fix this issue. I still haven't updated the tests for it yet, but it's a start. https://github.com/cljdoc/cljdoc-analyzer/pull/40
So with this PR now merged, how long should I have to wait before I can rebuild the documentation for farolero?
Congrats @suskeyhose! Someone just needs to update cljdoc to point to your change in cljdoc-analyzer. Technically I think you have the power, but this is your first commit, so we shall help. @martinklepsch I see you merged @suskeyhose’s cljdoc-analyzer change, are you in progress in making it live? If you are busy, lemme know and I can do.
Alright, that makes sense.
I’m not in front of my computer right now but could do it tomorrow:)
I can do now… I’ll test local and then push. Letcha know when done.
Thanks!
Am experiencing what seem to be local technical difficulties in testing (I don’t think there is anything wrong with your change). Will ping back.
Thanks for the heads up
The technical problem was squarely between my ears. Have pushed change to cljdoc and it should automatically deploy after build/test.
Awesome, thanks for the quick turnaround on this!
Happened to be available, so happy to help.
Ok your change is live!
Feel free to hit the hidden rebuild link to force a rebuild of your docs.
Thanks! It worked like a charm. 🙂
Awesome, thanks for the fix!
No problem! I hope to contribute more in future if I see any more issues.
That’s great! Welcome aboard!