Skip to content

Return an empty list when no std help --find results are found#13160

Merged
fdncred merged 1 commit intonushell:mainfrom
NotTheDr01ds:no-help-result-error
Jun 15, 2024
Merged

Return an empty list when no std help --find results are found#13160
fdncred merged 1 commit intonushell:mainfrom
NotTheDr01ds:no-help-result-error

Conversation

@NotTheDr01ds
Copy link
Copy Markdown
Contributor

Description

Fixes #13143 by returning an empty list when there are no results found by std help --find/-f

User-Facing Changes

In addition, prints a message to stderr.

Tests + Formatting

  • 🟢 toolkit fmt
  • 🟢 toolkit clippy
  • 🟢 toolkit test
  • 🟢 toolkit test stdlib

After Submitting

@fdncred fdncred merged commit 3a6d8aa into nushell:main Jun 15, 2024
@fdncred
Copy link
Copy Markdown
Contributor

fdncred commented Jun 15, 2024

sounds good. thanks

@hustcer hustcer added this to the v0.95.0 milestone Jun 15, 2024
132ikl pushed a commit to 132ikl/nushell that referenced this pull request Jun 20, 2024
…hell#13160)

# Description

Fixes nushell#13143 by returning an empty list when there are no results found
by `std help --find/-f`

# User-Facing Changes

In addition, prints a message to stderr.

# Tests + Formatting

- 🟢 `toolkit fmt`
- 🟢 `toolkit clippy`
- 🟢 `toolkit test`
- 🟢 `toolkit test stdlib`

# After Submitting
<!-- If your PR had any user-facing changes, update [the
documentation](https://github.com/nushell/nushell.github.io) after the
PR is merged, if necessary. This will help us keep the docs up to date.
-->
@fdncred fdncred added deprecated:pr-commands (deprecated: too vague) This PR changes our commands in some way notes:fixes Include the release notes summary in the "Bug fixes" section labels Jun 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deprecated:pr-commands (deprecated: too vague) This PR changes our commands in some way notes:fixes Include the release notes summary in the "Bug fixes" section

Projects

None yet

Development

Successfully merging this pull request may close these issues.

std help -f errors when there are no results

3 participants