Skip to content

Core library support in the language server.#1065

Merged
g-r-a-n-t merged 1 commit intoargotorg:masterfrom
g-r-a-n-t:lsp-core
Mar 15, 2025
Merged

Core library support in the language server.#1065
g-r-a-n-t merged 1 commit intoargotorg:masterfrom
g-r-a-n-t:lsp-core

Conversation

@g-r-a-n-t
Copy link
Copy Markdown
Collaborator

@g-r-a-n-t g-r-a-n-t commented Mar 7, 2025

Using rust-embed and moved static core setup to common::input. Atm users get an error when goto selecting something in the core library.

@g-r-a-n-t g-r-a-n-t force-pushed the lsp-core branch 2 times, most recently from bea3611 to 97fd1e0 Compare March 7, 2025 00:10
@g-r-a-n-t g-r-a-n-t marked this pull request as draft March 7, 2025 00:19
@g-r-a-n-t g-r-a-n-t marked this pull request as ready for review March 10, 2025 19:43
@g-r-a-n-t g-r-a-n-t force-pushed the lsp-core branch 2 times, most recently from 99d6279 to 0ab6e2f Compare March 12, 2025 21:08
@g-r-a-n-t g-r-a-n-t changed the title Lsp core Core library support in the language server. Mar 13, 2025
Copy link
Copy Markdown
Collaborator

@sbillig sbillig left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm fine with merging this as-is, but I think we need to be more careful with InputIngot::core (at least when we have multiple ingots in a single program). Each call to InputIngot::core returns a new InputIngot, so if we create two LocalIngotContexts via LocalIngotContext::new, they'll have different instances of the core ingot, and there will be two incompatible instances of the Option type, for example.

@g-r-a-n-t g-r-a-n-t merged commit 2263dc7 into argotorg:master Mar 15, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants