-
Notifications
You must be signed in to change notification settings - Fork 96
Closed
Labels
good first issueGood for newcomersGood for newcomers
Description
We should try augmenting the Window interface instead of using var: https://www.perplexity.ai/search/find-some-documentation-on-why-28TclBZATv2iVaO2wsbf1g
obsidian-mcp-tools/packages/shared/src/types/plugin-smart-connections.ts
Lines 99 to 102 in 72beac2
| declare global { | |
| // eslint-disable-next-line no-var | |
| var SmartSearch: SmartSearch | undefined; | |
| } |
From obsidianmd/obsidian-releases#5060 (comment):
You should change all instances of var to either const or let. var has function-level scope, so it can easily lead to bugs if you're not careful. Here's a pretty good guide with examples of why not to use var: why var is obsolete
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
good first issueGood for newcomersGood for newcomers