Skip to content

lsp and --ide-check fix for path self related diagnostics#14538

Merged
fdncred merged 1 commit intonushell:mainfrom
Bahex:ide-path-self-fix
Dec 7, 2024
Merged

lsp and --ide-check fix for path self related diagnostics#14538
fdncred merged 1 commit intonushell:mainfrom
Bahex:ide-path-self-fix

Conversation

@Bahex
Copy link
Copy Markdown
Member

@Bahex Bahex commented Dec 7, 2024

Description

fixes this where lsp and ide integration would produce the following error

nu --ide-check 100 "/path/to/env.nu"

with

const const_env = path self

would lead to

Error: nu::shell::file_not_found

  × File not found
   ╭─[/path/to/env.nu:1:19]
 1 │ const const_env = path self
   ·                   ────┬────
   ·                       ╰── Couldn't find current file
   ╰────

Tests + Formatting

  • 🟢 cargo fmt --all
  • 🟢 cargo clippy --workspace

@fdncred
Copy link
Copy Markdown
Contributor

fdncred commented Dec 7, 2024

Makes sense to me. Thanks

@fdncred fdncred merged commit 69fbfb9 into nushell:main Dec 7, 2024
@github-actions github-actions bot added this to the v0.101.0 milestone Dec 7, 2024
@fdncred fdncred added the A:LSP Language Server Protocol (nu-lsp) label Jan 11, 2025
@Bahex Bahex deleted the ide-path-self-fix branch March 22, 2026 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

A:LSP Language Server Protocol (nu-lsp)

Projects

None yet

Milestone

v0.101.0

Development

Successfully merging this pull request may close these issues.

2 participants

@Bahex @fdncred