Skip to content

fix: input listen's output type#18535

Merged
Bahex merged 1 commit into
nushell:mainfrom
Bahex:push-tuzzsxpyxlsy
Jul 6, 2026
Merged

fix: input listen's output type#18535
Bahex merged 1 commit into
nushell:mainfrom
Bahex:push-tuzzsxpyxlsy

Conversation

@Bahex

@Bahex Bahex commented Jul 6, 2026

Copy link
Copy Markdown
Member

Description

input listen command's signature output type does not match it's actual return value.

The command was added in 2bb0c1c, and apparently it's declared signature and actual output never matched.

User-facing changes (Release notes)

  • Fixed input listen's output type signature to match actual returned value.
    With the incorrect signature (and enforce-runtime-annotations on) let x = input listen would raise an error due to $x's inferred type (based on input listen's signature) not matching the value assigned to it.

@Bahex Bahex added notes:fixes Noted in "Bug fixes" section notes:ready Indicates Ready for Release notes labels Jul 6, 2026
@Bahex Bahex merged commit d5ac4c6 into nushell:main Jul 6, 2026
18 checks passed
@github-actions github-actions Bot added this to the v0.115.0 milestone Jul 6, 2026
nidara-duo pushed a commit to nidara-duo/nushell that referenced this pull request Jul 11, 2026
## Description

- brought up in: nushell#18528

`input listen` command's signature output type does not match it's
actual return value.

The command was added in 2bb0c1c, and
apparently it's declared signature and actual output never matched.

## User-facing changes (Release notes)

- Fixed `input listen`'s output type signature to match actual returned
value.
With the incorrect signature (and `enforce-runtime-annotations` on) `let
x = input listen` would raise an error due to `$x`'s inferred type
(based on `input listen`'s signature) not matching the value assigned to
it.
@cptpiepmatz cptpiepmatz modified the milestones: v0.115.0, v0.114.1 Jul 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

notes:fixes Noted in "Bug fixes" section notes:ready Indicates Ready for Release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants