Update Nushell init script for v0.105.0#2796
Merged
rsteube merged 1 commit intocarapace-sh:masterfrom Jun 4, 2025
Merged
Conversation
sholderbach
pushed a commit
to nushell/nushell
that referenced
this pull request
Jun 4, 2025
#15654 (#15806) # Description Adds a temporary workaround to prevent #15654 from being a breaking change when using a closure stored in a variable, and issues a warning. Also has a special case related to carapace-sh/carapace-bin#2796 which suggests re-running `carapace init`   # After Submitting Remove variable name detection after grace period
Contributor
Author
|
@rsteube Our release will be on the 10th, so it's probably safe to merge this in soon. Thanks 😄 |
rsteube
approved these changes
Jun 4, 2025
kumarUjjawal
pushed a commit
to kumarUjjawal/nushell
that referenced
this pull request
Jun 5, 2025
nushell#15654 (nushell#15806) # Description Adds a temporary workaround to prevent nushell#15654 from being a breaking change when using a closure stored in a variable, and issues a warning. Also has a special case related to carapace-sh/carapace-bin#2796 which suggests re-running `carapace init`   # After Submitting Remove variable name detection after grace period
12 tasks
12 tasks
This was referenced Aug 6, 2025
Closed
13 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is a preemptive PR to account for a breaking change which will be part of the next Nushell version, following the merge of nushell/nushell#15654. The change will be part of the v0.105.0 release, which will be on June 10th. This shouldn't be merged until then (or perhaps it could be merged early if carapace will have a release right before June 10th).
The default command now runs closures in order to support lazy evaluation, so to return a closure with
defaultyou must put the closure in the closure now. I've updated the Nushell init file to account for this.