Skip to content

fix: unexpected spaces at token head#214

Merged
fdncred merged 3 commits intonushell:mainfrom
blindFS:optim
Jul 8, 2025
Merged

fix: unexpected spaces at token head#214
fdncred merged 3 commits intonushell:mainfrom
blindFS:optim

Conversation

@blindFS
Copy link
Contributor

@blindFS blindFS commented Jul 8, 2025

This PR:

  1. reduces case count Tracking issue: performance optimization #185 : 3376 -> 3146, WASM comp time: 3m40 -> 2m20
  2. accidentally fixes the space issue of Assignment rhs includes all the white space after the assignment operator. #198 / hide-env / flags

@fdncred
Copy link
Contributor

fdncred commented Jul 8, 2025

interesting. thanks! that's a pretty significant reduction.

@fdncred fdncred merged commit 6810930 into nushell:main Jul 8, 2025
4 checks passed
@blindFS blindFS deleted the optim branch August 10, 2025 17:53
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.

Assignment rhs includes all the white space after the assignment operator.

2 participants