Skip to content

WIP: attemp to fix 149#163

Closed
asakura wants to merge 1 commit into
nushell:mainfrom
asakura:fix/issue-149
Closed

WIP: attemp to fix 149#163
asakura wants to merge 1 commit into
nushell:mainfrom
asakura:fix/issue-149

Conversation

@asakura

@asakura asakura commented Mar 30, 2026

Copy link
Copy Markdown
Contributor

#149

The core issue is that nu_parser attempts to compile a block while parsing. Because it has already accumulated ParseErrors::ModuleNotFound and ParseErrors::VariableNotFound errors in the working set, the compilation fails and barks to stderr. As a result, nufmt never gets the chance to filter out these parsing errors.

at /home/asakura/.cargo/registry/src/index.crates.io-1949cf8c6b5b557f/nu-parser-0.110.0/src/parser.rs:6748:18
1: nu_parser::parse_keywords::parse_def_inner
at /home/asakura/.cargo/registry/src/index.crates.io-1949cf8c6b5b557f/nu-parser-0.110.0/src/parse_keywords.rs:649:25
2: nu_parser::parse_keywords::parse_def
at /home/asakura/.cargo/registry/src/index.crates.io-1949cf8c6b5b557f/nu-parser-0.110.0/src/parse_keywords.rs:455:24
3: nu_parser::parser::parse_builtin_commands
at /home/asakura/.cargo/registry/src/index.crates.io-1949cf8c6b5b557f/nu-parser-0.110.0/src/parser.rs:6195:19
4: nu_parser::parser::parse_pipeline
at /home/asakura/.cargo/registry/src/index.crates.io-1949cf8c6b5b557f/nu-parser-0.110.0/src/parser.rs:6640:9
5: nu_parser::parser::parse_block
at /home/asakura/.cargo/registry/src/index.crates.io-1949cf8c6b5b557f/nu-parser-0.110.0/src/parser.rs:6674:24
6: nu_parser::parser::parse
at /home/asakura/.cargo/registry/src/index.crates.io-1949cf8c6b5b557f/nu-parser-0.110.0/src/parser.rs:7258:22
7: nu_formatter::formatting::format_inner_with_options

@asakura asakura changed the title WIP: feat: Consolidate Nix flake check into nix.yml workflow WIP: attemp to fix 149 Mar 30, 2026
@fdncred

fdncred commented May 18, 2026

Copy link
Copy Markdown
Contributor

Still working on this?

@fdncred

fdncred commented May 20, 2026

Copy link
Copy Markdown
Contributor

no response

@fdncred fdncred closed this May 20, 2026
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