Skip to content

internal: Fix lsp_ext field name for RecursiveMemoryLayoutNode#21986

Merged
ChayimFriedman2 merged 1 commit intorust-lang:masterfrom
A4-Tacks:lspext-defs-layout-node
Apr 9, 2026
Merged

internal: Fix lsp_ext field name for RecursiveMemoryLayoutNode#21986
ChayimFriedman2 merged 1 commit intorust-lang:masterfrom
A4-Tacks:lspext-defs-layout-node

Conversation

@A4-Tacks
Copy link
Copy Markdown
Member

@A4-Tacks A4-Tacks commented Apr 7, 2026

Due editors/code/src/commands.ts uses inline HTML, so this error pass to type check

Due `editors/code/src/commands.ts` uses inline HTML, so this error pass to type check
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 7, 2026
@ChayimFriedman2 ChayimFriedman2 added this pull request to the merge queue Apr 9, 2026
@ChayimFriedman2
Copy link
Copy Markdown
Contributor

I don't like the usage of inner HTML including JavaScript, it's better to remove it.

Merged via the queue into rust-lang:master with commit c3a4b78 Apr 9, 2026
18 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 9, 2026
@A4-Tacks
Copy link
Copy Markdown
Member Author

A4-Tacks commented Apr 9, 2026

it's better to remove it.

I am not familiar with HTML and JavaScript

@ChayimFriedman2
Copy link
Copy Markdown
Contributor

Fine, it was not directed at you specifically.

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.

3 participants