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

Don't merge it....try to update nushell on latest main #61

Closed
wants to merge 1 commit into from

Conversation

WindSoilder
Copy link

@WindSoilder WindSoilder commented Apr 28, 2024

An error happened when trying to update dependencies to latest main:

❯ open Cargo.toml | nu_plugin_explore
Error:   × Can't start nu_plugin_explore
   ╭─[entry #8:1:19]
 1 │ open Cargo.toml | nu_plugin_explore
   ·                   ────────┬────────
   ·                           ╰── must run in a terminal
   ╰────
  help: ensure that you are running in a terminal, and that the plugin is not communicating over stdio

@amtoine
Copy link
Owner

amtoine commented Apr 28, 2024

building the plugin and Nushell to try repro this... ♻️

@amtoine
Copy link
Owner

amtoine commented Apr 28, 2024

so it's not related to upgrading Nushell itself, just bumping the deps as you did is enough to reproduce this issue.

@WindSoilder
Copy link
Author

Closing it infavor of #63 . Thanks @devyn for looking into this!

@amtoine amtoine closed this May 4, 2024
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.

None yet

2 participants