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.

Sign up for GitHub

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

Jump to bottom

Type check commands #67

Closed
sophiajt opened this issue Jun 2, 2019 · 1 comment
Closed

Type check commands #67

sophiajt opened this issue Jun 2, 2019 · 1 comment

Comments

@sophiajt
Copy link
Contributor

sophiajt commented Jun 2, 2019

We have enough information about the commands and the piped chain of commands that we could do some simple typechecking on it before we run it.

Some examples:

  • Commands that expect objects need objects to flow in
  • Commands that expect strings can't have objects flow in
  • External commands that use $it expect strings
@sophiajt
Copy link
Contributor Author

We've added the support to do this and have laid in the groundwork for more, so going ahead and closing this

elferherrera pushed a commit to elferherrera/nushell that referenced this issue Feb 7, 2022
@sophiajt
Merge pull request nushell#67 from nushell/revert-65-list_table
3421a8b 
Revert "improve table for lists"
Hofer-Julian pushed a commit to Hofer-Julian/nushell that referenced this issue Jan 27, 2023
Merge pull request nushell#67 from janosimas/master
ffeb566 
Replace splitting tag in git log
amtoine added a commit to amtoine/nushell that referenced this issue Apr 6, 2023
@amtoine
fix all mentions to the old path to the standard library
e251632 
Command used
```bash
[.github/**/* **/*.rs **/*.nu]
| each {|target|
    sd 'crates/nu-utils/standard_library' 'crates/standard_library' $target
}
```

i used this complex command because of
```bash
>_ sd 'crates/nu-utils/standard_library' 'crates/standard_library' **/*
Error: nu::shell::external_command

  × External command failed
   ╭─[entry nushell#67:1:1]
 1 │ sd 'crates/nu-utils/standard_library' 'crates/standard_library' **/*
   · ─┬
   ·  ╰── can't run executable
   ╰────
  help: Argument list too long (os error 7)
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@sophiajt