Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

from ini doesn't work as documentation says #1091

Closed
arsfeld opened this issue Oct 2, 2023 · 4 comments · Fixed by #1199
Closed

from ini doesn't work as documentation says #1091

arsfeld opened this issue Oct 2, 2023 · 4 comments · Fixed by #1199
Labels
documentation Improvements or additions to documentation

Comments

@arsfeld
Copy link

arsfeld commented Oct 2, 2023

Describe the bug

The documentation mentions a command called from ini, but it doesn't work unless you install a custom plugin, which is nowhere to be found except in a hidden away release note.

This was done in nushell/nushell#7942 but it seems the documentation has not been updated.

How to reproduce

  1. Install nushell
  2. Use the from ini command
  3. Get error mentioning extra positional argument

Expected behavior

Expected from ini to work as documented

Screenshots

No response

Configuration

key value
version 0.85.0
branch
commit_hash
build_os linux-x86_64
build_target x86_64-unknown-linux-gnu
rust_version rustc 1.72.0 (5680fa18f 2023-08-23) (Homebrew)
cargo_version cargo 1.72.0 (26bba4830 2023-08-26)
build_time 2023-09-19 18:42:47 +00:00
build_rust_channel release
allocator mimalloc
features dataframe, default, sqlite, trash, which, zip
installed_plugins

Additional context

No response

@fdncred
Copy link
Collaborator

fdncred commented Oct 2, 2023

I'm not surprised our documentation is out of date. I'm glad you found it in nu_plugin_formats. We'd accept a PR to update the documentation. I'll probably move this to the docs repo.

Thanks for filing the issue!

@fdncred fdncred added the documentation Improvements or additions to documentation label Oct 2, 2023
@fdncred fdncred transferred this issue from nushell/nushell Oct 2, 2023
@TWSiO
Copy link
Contributor

TWSiO commented Jan 4, 2024

Should from ini and the other removed commands in that package be removed from the docs entirely, or should they have a warning or danger callout (custom container) mentioning that they used to be included but are now in that plulgin?

@fdncred
Copy link
Collaborator

fdncred commented Jan 4, 2024

Probably a notice to say that you have to compile the plugin for the commands in nu_plugin_formats to work.

@arsfeld
Copy link
Author

arsfeld commented Jan 7, 2024

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants