This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2023-02-10
Channels
- # announcements (6)
- # babashka (38)
- # beginners (85)
- # biff (3)
- # calva (2)
- # cider (11)
- # clerk (14)
- # clj-kondo (6)
- # clj-on-windows (27)
- # clj-together (2)
- # cljsrn (18)
- # clojure (106)
- # clojure-austin (1)
- # clojure-belgium (1)
- # clojure-europe (19)
- # clojure-nl (1)
- # clojure-norway (9)
- # clojure-uk (2)
- # clr (2)
- # cryogen (1)
- # cursive (10)
- # datahike (3)
- # datavis (2)
- # datomic (15)
- # emacs (7)
- # graalvm (10)
- # graphql (20)
- # gratitude (1)
- # hyperfiddle (1)
- # improve-getting-started (23)
- # joyride (24)
- # london-clojurians (1)
- # lsp (22)
- # malli (4)
- # matcher-combinators (3)
- # membrane (13)
- # off-topic (1)
- # pathom (24)
- # polylith (9)
- # react (31)
- # reagent (9)
- # releases (1)
- # remote-jobs (1)
- # reveal (3)
- # shadow-cljs (50)
- # spacemacs (3)
- # specter (5)
- # xtdb (5)
Is displaying call hierarchy (i.e. the tree of functions that call a function) something LSP could/should do?
If using lsp-mode lsp-treemacs-call-hierarchy
, I think it works in vscode as well.
https://clojure-lsp.io/features/#call-hierarchy
lsp-treemacs-call-hierarchy
for incoming call hierarchy
C-u
lsp-treemacs-call-hierarchy
for outgoing
VS Code. I see Code commands include “Calls: Show call hierarchy” but that renders no results, and possibly has nothing to do with LSP. @U0ETXRFEW?
Hm, is there some simple way to exclude calls from test files / path?
For the record: The Show Call Hierarchy command as such is built-in to VS Code. Needs to be implemented by the language extension. clojure-lsp implements it in the case of Calva.
I think that should be done for each client @U0522TWDA, not sure if vscode supports that but I can see how convenient it is, I used that in Eclipse a lot in the past
I don't think VS Code enables a way to excludes calls from certain files. Might be something Calva can be made to do, idk.
go to definition doesn't seem to work if a namespace uses load
. Is this on the roadmap? Should I open a github issue about it?
record?
is in clojure.core, but the definition can't be found cuz it's implemented in core_deftype.clj
: https://github.com/clojure/clojure/blob/clojure-1.11.1/src/clj/clojure/core_deftype.clj#L406
2023-02-10T14:27:17.931Z DEBUG [clojure-lsp.server:55] - [Trace - 2023-02-10T14:27:17.931Z] Received request 'textDocument/definition - (95)'
Params: {
"text-document" : {
"uri" : "file:///home/noah/personal/spat/src/noahtheduke/core_extensions.clj"
},
"position" : {
"line" : 15,
"character" : 4
}
}
2023-02-10T14:27:17.932Z INFO [clojure-lsp.handlers:246] - :definition 0ms
2023-02-10T14:27:17.933Z DEBUG [clojure-lsp.server:55] - [Trace - 2023-02-10T14:27:17.932Z] Sending response 'textDocument/definition - (95)'. Request took 1ms.
Result: null
I've only seen load
used in clojure implementation libraries. Adding support for that seems rather niche
🤷 that's why i'm asking
a long time ago, we used load in a project that put all of a game's engine logic into a single namespace but split across many many files. was a pain in the butt to work with lol