profile
viewpoint
Jonathan Turner jonathandturner @Microsoft New Zealand jonathanturner.org Current: @azure and @nushell. Previously: @rust-lang, PM for TypeScript @microsoft.

jonathandturner/AudioScope-ng2 35

AudioScope demo written in Angular 2 + TypeScript

jonathandturner/actress 3

Experimenting with a simple actor system in Rust

jonathandturner/api-extractor.com-website 0

GitHub Pages website for api-extractor.com

jonathandturner/azure-functions-rs 0

Create Azure Functions with Rust!

jonathandturner/azure-rest-api-specs 0

The source for REST API specifications for Microsoft Azure.

jonathandturner/azure-sdk-for-js 0

An isomorphic javascript sdk for Azure services

jonathandturner/blog.rust-lang.org 0

The Rust Programming Language Blog

jonathandturner/book 0

The Rust Programming Language

push eventjonathandturner/azure-sdk-for-js

Jonathan Turner

commit sha 8136ce164d1cb35778693a8158a3de52cf9544a9

switched to managed id

view details

push time in 2 hours

push eventjonathandturner/azure-sdk-for-js

Jonathan Turner

commit sha 8cb75cddc1454081d7f63458f1fe035179601242

More improvements

view details

Jonathan Turner

commit sha dd4ed4ad5fa1661525bd5984651ca1dd05017ed9

include missing file and bump script

view details

push time in 2 hours

push eventjonathandturner/azure-sdk-for-js

Jonathan Turner

commit sha e89a140c83a53968ac2f70c20c3e11fd6777c5f1

More improvements

view details

push time in 4 hours

push eventjonathandturner/azure-sdk-for-js

Jonathan Turner

commit sha ee03ea7c48766ececb9142dd93d29e5526a048df

More improvements

view details

push time in 4 hours

push eventjonathandturner/azure-sdk-for-js

Jonathan Turner

commit sha 6b355c22b7006f448418843bb9dbe75d9925c179

More improvements

view details

push time in 5 hours

push eventjonathandturner/azure-sdk-for-js

Jonathan Turner

commit sha cae9e3657f50ffa1013ce4da8dd00b016bfebefa

More improvements

view details

push time in 5 hours

push eventjonathandturner/azure-sdk-for-js

Jonathan Turner

commit sha d8906a1a8f8f352d87d3d8c08473b9be08ad8585

More improvements

view details

push time in a day

push eventjonathandturner/azure-sdk-for-js

Jonathan Turner

commit sha abce98a87f6d3efbc66988dee050c637074c8186

More improvements

view details

push time in a day

push eventjonathandturner/azure-sdk-for-js

Jonathan Turner

commit sha eaaa7cf5a230e035449bf5d91daf5d07ed461768

More improvements

view details

push time in a day

push eventjonathandturner/azure-sdk-for-js

Jonathan Turner

commit sha e75e7e56a269206e90c300ac7a0a4078963184b0

More improvements

view details

push time in a day

push eventjonathandturner/azure-sdk-for-js

Jonathan Turner

commit sha a1afc2a0dbe90705f19d575725502cd69274e48f

More improvements

view details

push time in a day

push eventjonathandturner/azure-sdk-for-js

Jonathan Turner

commit sha 36e312b65fa8dccd4e0970c8e24dfe5459dfbe38

More improvements

view details

push time in a day

push eventjonathandturner/azure-sdk-for-js

Jonathan Turner

commit sha 4ebd81fb1d30ec6cce179840dd4d27791bb4276d

More improvements

view details

push time in a day

push eventjonathandturner/azure-sdk-for-js

Jonathan Turner

commit sha bf71d5e1ede3738e6da876e9ed2b96ef4d261bb2

Improve k8s test

view details

push time in a day

push eventjonathandturner/rhai

Stephen Chung

commit sha 829621a3d4b91e6857f4f6dfa033046b18479cdc

Trap array out-of-bound index access error.

view details

Stephen Chung

commit sha b2747b076e845d5041783e5ded000d657ea8c4da

Add bouncs check to array index access.

view details

Stephen Chung

commit sha 157aa2be948e723d23f1b24283a22c84c89c45a1

Add for statement and range function; New register_box_fn for functions returning Box<dyn Any> (which should not be boxed again); Register standard utility functions in library (e.g. print, string concat).

view details

Stephen Chung

commit sha 896579869c714bdca51538237897f126817104fe

Fix test_mismatched_op.

view details

Stephen Chung

commit sha c58b3e644a51b7eb72eade2be0b1c27c0ce4622d

Ignore .vscode

view details

Stephen Chung

commit sha b3247204c6626e06a660fba464bfe43c15a61435

Include filename in ErrorCantOpenScriptFile.

view details

Stephen Chung

commit sha e4e471fd208d1a631ca05b8473623af63d7dbe55

Add line/position in compilation errors; Add compile to AST; FIx infinite loop in close parens.

view details

Jonathan Turner

commit sha 2a663d5c07469eb533e5982aaf9f539cc1289413

Merge pull request #97 from schungx/master Handle arrays properly, plus other enhancements.

view details

push time in a day

PR merged jonathandturner/rhai

Handle arrays properly, plus other enhancements.
+1006 -479

2 comments

10 changed files

schungx

pr closed time in a day

pull request commentjonathandturner/rhai

Handle arrays properly, plus other enhancements.

@schungx this looks great! Thanks for all the effort you put into it.

schungx

comment created time in a day

issue commentnushell/contributor-book

.Net Core Plugin Example

@myty sounds great! Feel free to reach out here or on discord if you have any questions.

myty

comment created time in a day

issue commentnushell/nushell

Proposal: uncouple table stages in the output stream

@thegedge - good question... I'm starting to wonder if we should simplify a bit. Right now the reading would be similar to SAX, in that the reader would need to hold more state to parse as values came through the pipeline.

Maybe we need discrete and begin/end versions of nodes. That would let you have something much closer to the current protocol if you knew all your values were discrete.

Will think about it a bit more to see if that can be simplified any.

jonathandturner

comment created time in a day

push eventnushell/nushell

Ryan Blecher

commit sha a86a0abb906af40aae97b72999cb6d8a8f31a412

Plugin documentation (#1431) * Add very basic documentation. Need to play with rest of the api to figure out what it does * Add some documentation to more of the Plugin API methods * fmt

view details

push time in 2 days

PR merged nushell/nushell

Plugin documentation

Add documentation to methods in the Plugin trait.

+14 -0

2 comments

1 changed file

notryanb

pr closed time in 2 days

pull request commentnushell/nushell

Plugin documentation

Thanks!

notryanb

comment created time in 2 days

push eventjonathandturner/azure-sdk-for-js

Jonathan Turner

commit sha 72f2005a30cb51095173c4e0a565c6363d79b187

Add Kubernetes early draft

view details

push time in 2 days

issue openednushell/nushell

Proposal: uncouple table stages in the output stream

Is your feature request related to a problem? Please describe. The current protocol for output streams is pretty powerful. We can describe streams of rows in a way that can be consumed at different stages in the pipeline, and in a structural way. That said, the current system has a few drawbacks:

  • There is no way to stream non-table data. Eg) streaming of large binaries or large strings is not currently possible as they assume you have the whole data unit and can send as a single unit (rather than stream)
  • There is no way to stream inner table data. Inner tables must be fully realized as a single unit
  • Rows in the table currently contain column names, which leads to lots of duplication
  • It's currently ambiguous if the user means rows of strings (eg what comes from lines) and the contents of a text file
  • Related: we assume that contents streams out of an external command as text with line-endings. This feature would allow us to express a stream of bytes, which the user could then optionally turn into lines

Describe the solution you'd like

The solution would be in two parts:

Un-coupling the table stages themselves

Currently we have Row and Table as two Value types, but this isn't expressive enough.

I suggest we expand this into the actual stages:

  • BeginTable / EndTable
  • Header
  • BeginRow / EndRow
  • BeginCell / EndCell (for discrete cells we could also have a shortcut Cell)

Where we have:

  • Row (header1: data1_1, header2: data1_2, ...)
  • Row (header1, data2_1, header2: data2_2, ...)

We would have:

  • BeginTable
  • Header (header1, header2, ...)
  • BeginRow
  • Cell(data1_1)
  • Cell(data1_2)
  • EndRow
  • BeginRow
  • Cell(data2_1)
  • Cell(data2_2)
  • EndRow
  • EndTable

Uncoupling tables from other value types in the stream

Each value that comes in the stream over the first value is assume to be rows of a table, but this prevents us from representing streaming other kinds of data. I propose we no longer assume that data units are rows. They must be expressed as part of the row data in the protocol, or we assume they're part of a stream of bytes or text.

Describe alternatives you've considered We could switch to a more command-based protocol that, like an instruction set around how to build up a table. This would be powerful, but I feel the value-based approach should be explore more before we investigate this direction.

Drawbacks to this proposal There are drawbacks to this proposal. The new protocol is more verbose, which increases wire and memory traffic and may impact performance. It also makes emitting tables more complex in that you have more steps - this can be mitigated somewhat with convenience functions. I believe this complexity is worthwhile, as it opens up the possibility to express different kinds of streams that were not expressible in the current design.

created time in 2 days

push eventnushell/contributor-book

Jonathan Turner

commit sha 06d5bc13d0e51f5475e89a0ba65f58c5262aa8e1

Update plugins.md

view details

push time in 2 days

push eventnushell/contributor-book

Jonathan Turner

commit sha 75e113abf3aaf32c583be375b61aa9af58278bb8

Update plugins.md

view details

push time in 2 days

push eventnushell/contributor-book

Jonathan Turner

commit sha 2c2162a1501a064bdc1b4f3e62c856ff97a592e5

Update plugins.md

view details

push time in 2 days

issue commentnushell/contributor-book

.Net Core Plugin Example

Thanks for sharing!

I'm happy to add a link to your project for C# developers.

In the future, we'll definitely want to grow this to have a section for each language in the documentation that helps walk you through creating plugins. This will likely happen after the protocol calms down a bit more (and we get closer to 1.0).

myty

comment created time in 2 days

delete branch jonathandturner/crossterm

delete branch : fix_raw_rd

delete time in 3 days

PR opened crossterm-rs/crossterm

Reviewers
Automatically close raw fd when finished using

Getting the terminal size will leak a file descriptor currently. (https://github.com/crossterm-rs/crossterm/issues/382)

This uses the auto-closing logic to avoid this issue.

+6 -6

0 comment

2 changed files

pr created time in 4 days

issue openedcrossterm-rs/crossterm

Getting the terminal size leaks fd in unix

Describe the bug If you keep getting the terminal size, you'll leak enough file descriptors that the program will rash

To Reproduce Steps to reproduce the behavior:

  1. A tight loop where you check for the terminal size

Expected behavior The file descriptors should get automatically close when you're finished them.

OS Unix

Terminal/Shell Bash/Nushell

created time in 4 days

create barnchjonathandturner/crossterm

branch : fix_raw_rd

created branch time in 4 days

fork jonathandturner/crossterm

Cross platform terminal library rust

fork in 4 days

push eventnushell/nushell

Jonathan Turner

commit sha adcda450d5d63c52238c23354eb38a9b13d23f14

Update LICENSE

view details

push time in 5 days

issue closedstarship/starship

Spacing between characters in Windows Terminal seems too large/small

<!-- ───────────────────────────────────────────── ⚠️ IMPORTANT: Please run the following command to create an issue:

                                                                        starship bug-report

                                  An issue will be pre-populated with your system's configuration, 
                                                      making the process a whole lot quicker 😊

───────────────────────────────────────────── -->

Bug Report

Current Behavior

No space between 'master' and 'is', also 'preview' and 'via'. The other spacing looks a little large in parts.

Expected Behavior

Even spacing between colored text.

Additional context/Screenshots

image

Possible Solution

<!--- Only if you have suggestions on a fix for the bug -->

Environment

  • Starship version: [the output of starship --version] 0.35.1
  • Shell type: [fish, zsh] nushell
  • Shell version: [the output of fish --version or zsh --version] 0.9.1
  • Terminal emulator: [e.g. iTerm, Hyper, Terminator]: Windows Terminal (Preview), Version: 0.8.10261.0
  • Operating system: [e.g. macOS 10.13.4, Windows 10]: Windows 10

Relevant Shell Configuration

Default configuration from nushell

Starship Configuration

<!-- Can be found in ~/.config/starship.toml -->

Default configuration from nushell

closed time in 5 days

jonathandturner

issue commentstarship/starship

Spacing between characters in Windows Terminal seems too large/small

Confirmed fixed now. Thanks

jonathandturner

comment created time in 5 days

push eventjonathandturner/azure-sdk-for-js

Jonathan Turner

commit sha 0a95ad150ec44b55a1415c67174320e72edd1cc9

Add drafts of AzureVM and Cloudshell

view details

push time in 6 days

push eventnushell/blog

Jonathan Turner

commit sha c49292734df28d70f9667ba5c24204b77d941f86

Update _config.yml

view details

push time in 6 days

pull request commentnushell/nushell

Implements --get, --set, and --remove for `env`

First off, thanks for this. Much appreciated.

I hate to say it, but this functionality is already in Nu, though it's a bit harder to find. You can update the environment variables using the technique in https://www.nushell.sh/blog/2019/11/05/nushell-0_5_0.html#adding-variables-to-your-environment. It's a bit wordy now, but we'll be adding aliases soon that should help make it quite a bit smaller.

One thing that will probably fall out from that is moving away from the env command towards using other techniques.

pwoolcoc

comment created time in 7 days

push eventnushell/nushell

Corvus Corax

commit sha e38442782efd76ae606709ee257b99931c80ce1f

Command documentation for `du` (#1416)

view details

push time in 7 days

PR merged nushell/nushell

Command documentation for `du`

@coolshaurya, documentation for du

+26 -0

2 comments

1 changed file

Amanita-muscaria

pr closed time in 7 days

pull request commentnushell/nushell

Command documentation for `du`

Nice

Amanita-muscaria

comment created time in 7 days

push eventnushell/nushell

Sean Hellum

commit sha b98f89321773548b81d803e44494bec831df5428

add a `touch` command (#1399)

view details

push time in 7 days

PR merged nushell/nushell

Reviewers
add a `touch` command

Add a touch command!

Fixes #1357

+78 -0

2 comments

5 changed files

JesterOrNot

pr closed time in 7 days

issue closednushell/nushell

Add touch command

Since there are already implementations of some commands such as mv cp etc, I think it would be also great to add the touch command. Even in it's most simple stage it should support:

  1. Creating an empty file (or even multiple empty files)
  2. Updating access and modification time of a file

Creating an empty file is a simple piece of code but changing the access-modification times are a little bit harder because Windows handles metadata differently than Linux. We can consider using the crate filetime since it's cross-platform and may make things easier.

closed time in 7 days

UltraWelfare

pull request commentnushell/nushell

add a `touch` command

Looks good!

JesterOrNot

comment created time in 7 days

push eventnushell/blog

Waldir Pimenta

commit sha 3355899823d827c35b4a02e21a296fe66ee4f94a

Fix typo in the 0.10.0 announcement post

view details

Waldir Pimenta

commit sha 8ae2354cf00c1ec3bfd5449514a81db95688272f

Update 2020-02-18-nushell-0_10_0.md

view details

Jonathan Turner

commit sha 459c31e7ace9f0738c3ba5357dfa3f5625b4f388

Merge pull request #21 from waldyrious/patch-2 Fix typos in the 0.10.0 announcement post

view details

push time in 7 days

PR merged nushell/blog

Fix typos in the 0.10.0 announcement post
+2 -2

1 comment

1 changed file

waldyrious

pr closed time in 7 days

pull request commentnushell/blog

Fix typos in the 0.10.0 announcement post

Thanks!

waldyrious

comment created time in 7 days

created tagnushell/nushell

tag0.10.0

A modern shell written in Rust

created time in 7 days

push eventnushell/blog

Jonathan Turner

commit sha 79fc6a21308a0bbbd10c4acd9ce08580a8be7c10

Add 0.10.0 release post

view details

push time in 7 days

Pull request review commentnushell/nushell

add a `touch` command

+use crate::prelude::*;+use nu_errors::ShellError;+use nu_protocol::{CallInfo, Signature, SyntaxShape, Value};+use nu_source::Tagged;+use std::error::Error;+use std::fs::OpenOptions;+use std::path::PathBuf;++pub struct Touch;++#[derive(Deserialize)]+pub struct TouchArgs {+    pub target: Tagged<PathBuf>,+}++impl PerItemCommand for Touch {+    fn name(&self) -> &str {+        "touch"+    }+    fn signature(&self) -> Signature {+        Signature::build("touch").required(+            "filename",+            SyntaxShape::Path,+            "the path of the file you want to create",+        )+    }+    fn usage(&self) -> &str {+        "creates a file"+    }+    fn run(+        &self,+        call_info: &CallInfo,+        _registry: &CommandRegistry,+        raw_args: &RawCommandArgs,+        _input: Value,+    ) -> Result<OutputStream, ShellError> {+        call_info+            .process(&raw_args.shell_manager, raw_args.ctrl_c.clone(), touch)?+            .run()+    }+}+fn touch(args: TouchArgs, _context: &RunnablePerItemContext) -> Result<OutputStream, ShellError> {+    match OpenOptions::new()

You can look through the tests we have now to get a feel for how they work. Really all you'll need to do is touch mytestfile and then make sure that the file exists after the test runs

JesterOrNot

comment created time in 8 days

delete branch jonathandturner/nushell

delete branch : bump_to_0_10_0

delete time in 8 days

push eventnushell/nushell

Jonathan Turner

commit sha 0f7c72367206124b8dc7be0d3685946b2fefc3e9

Bump version to 0.10.0 (#1403)

view details

push time in 8 days

PR merged nushell/nushell

Bump version to 0.10.0
+152 -152

1 comment

23 changed files

jonathandturner

pr closed time in 8 days

PR opened nushell/nushell

Bump version to 0.10.0
+152 -152

0 comment

23 changed files

pr created time in 8 days

create barnchjonathandturner/nushell

branch : bump_to_0_10_0

created branch time in 8 days

Pull request review commentnushell/nushell

add a `touch` command

+use crate::prelude::*;+use nu_errors::ShellError;+use nu_protocol::{CallInfo, Signature, SyntaxShape, Value};+use nu_source::Tagged;+use std::error::Error;+use std::fs::OpenOptions;+use std::path::PathBuf;++pub struct Touch;++#[derive(Deserialize)]+pub struct TouchArgs {+    pub target: Tagged<PathBuf>,+}++impl PerItemCommand for Touch {+    fn name(&self) -> &str {+        "touch"+    }+    fn signature(&self) -> Signature {+        Signature::build("touch").required(+            "filename",+            SyntaxShape::Path,+            "the path of the file you want to create",+        )+    }+    fn usage(&self) -> &str {+        "creates a file"+    }+    fn run(+        &self,+        call_info: &CallInfo,+        _registry: &CommandRegistry,+        raw_args: &RawCommandArgs,+        _input: Value,+    ) -> Result<OutputStream, ShellError> {+        call_info+            .process(&raw_args.shell_manager, raw_args.ctrl_c.clone(), touch)?+            .run()+    }+}+fn touch(args: TouchArgs, _context: &RunnablePerItemContext) -> Result<OutputStream, ShellError> {+    match OpenOptions::new()

Can you add a test that this creates a file?

JesterOrNot

comment created time in 8 days

pull request commentnushell/nushell

Primitive rows are always shown in the same table.

@andrasio I landed #1401, so we can close this and look for a different fix

andrasio

comment created time in 8 days

push eventnushell/nushell

Jonathan Turner

commit sha afce2fd0f9e8bc9a11ff938ee6fbd07bb4c57cfa

Revert "Display rows in the same table regardless of their column order given they are equal. (#1392)" (#1401) This reverts commit 4fd99742042233f001d9d1d34e85cd804669effa.

view details

push time in 8 days

PR merged nushell/nushell

Revert "Display rows in the same table regardless of their column order."

Reverts nushell/nushell#1392

This had unintended consequences. Since we want to release tomorrow, we'll revert for now and look into a better fix later.

+4 -42

1 comment

2 changed files

jonathandturner

pr closed time in 8 days

PR opened nushell/nushell

Revert "Display rows in the same table regardless of their column order."

Reverts nushell/nushell#1392

This had unintended consequences. Since we want to release tomorrow, we'll revert for now and look into a better fix later.

+4 -42

0 comment

2 changed files

pr created time in 8 days

create barnchnushell/nushell

branch : revert-1392-print-table

created branch time in 8 days

PR opened Azure/azure-sdk-for-js

Add web job manual integration test

DRAFT:

This adds a manual integration test for web jobs using a set of powershell commands described in the README. The associated .zip will create a webjob with the name AzureTestJob when uploaded. Needs a bit more testing before pulling off the draft label.

+121 -0

0 comment

2 changed files

pr created time in 8 days

create barnchjonathandturner/azure-sdk-for-js

branch : manual_integration_tests_1

created branch time in 8 days

issue openednushell/nushell

Allow pipes in strings

Describe the bug The pipe character (|) is currently not allowed in strings

To Reproduce Steps to reproduce the behavior:

  1. ❯ az webapp create -n "myapp" -g "mygroup" -p "myplan" -r "node|10.15"

Will give you:

'10.15' is not recognized as an internal or external command,
operable program or batch file.

Expected behavior The pipe character will be treated as part of the string

Configuration (please complete the following information):

  • OS: [e.g. Windows]: Window 10
  • Version [e.g. 0.4.0]: 0.9.1

created time in 8 days

issue commentnushell/nushell

Add touch command

@JesterOrNot please do :)

UltraWelfare

comment created time in 8 days

issue openednushell/nushell

Switch remove automatic line-ending parsing

Describe the bug We currently assume that the input stream from stdin wants to be parsed using line endings like \n but this is incorrect for a few reasons:

  • If the user wishes to work with the text in a different way, they don't have the control
  • If the stdin doesn't end with a line ending, we would fail to read it in. We currently have a workaround where we insert this line ending, but this is incorrect in the general case
  • If stdin is binary data, we might end up reading in bytes that aren't characters and look for line endings in them. This will cause panics and confusion

To Reproduce Steps to reproduce the behavior:

  1. Stream between an external and another command

Expected behavior The solution is in a few parts.

  • Make streaming from an external take in bytes (you can think of this as packets of bytes). These would be unaltered from what is read in from the external's stdout.
  • Give internal commands a way to read in a string of text
  • Give internal commands a way to collect the stream as text
    • Note: it's unclear if these should be automatic conversions based on the type of the input or manually done by each command
  • Remove the automatic line ending parsing (LineCodec), and move to BytesCodec
  • Next, when viewing, do not treat a stream of non-row values as separate rows. Instead, simply display the text (or binaryview the binary) using simple streaming methods Note: Do not collect text or binary and use in a viewer by default as these streams may not terminate

created time in 8 days

push eventnushell/cookbook

Sebastian Jung

commit sha 3337e75439ac04e81253e65e345cbd661a3e42e5

add table of contents

view details

Jonathan Turner

commit sha 666fd81ff3dca52cad091c4109a64fddbb581db1

Merge pull request #13 from sebastian-xyz/table-of-contents add table of contents

view details

push time in 8 days

PR merged nushell/cookbook

add table of contents

PR adds table of contents at the beginning of the book so people can easily jump to chapter.

+129 -92

1 comment

5 changed files

sebastian-xyz

pr closed time in 8 days

pull request commentnushell/cookbook

add table of contents

Oh nice!

sebastian-xyz

comment created time in 8 days

issue closednushell/nushell

Shorthand flags should support `-abc`

Is your feature request related to a problem? Please describe. more ergonomic to type -abc instead -a -b -c

Describe the solution you'd like Flags in the form -abc to be supported.

Describe alternatives you've considered none

Additional context Currently, doing -abc is the same as doing -a

closed time in 9 days

coolshaurya

issue commentnushell/nushell

Shorthand flags should support `-abc`

@coolshaurya - yeah, let's close this and use #1240 to track it.

coolshaurya

comment created time in 9 days

delete branch jonathandturner/nushell

delete branch : force_stdout_newline

delete time in 11 days

push eventnushell/nushell

Jonathan Turner

commit sha 9bc5022c9c0916b2c7f6be8dfcaafcfb30f2122f

Force a \n at the end of a stdout stream (#1391) * Force a \n at the end of a stdout stream * clippy

view details

push time in 11 days

issue closednushell/nushell

Nushell panics on output of external command without trailing newline

Describe the bug When the output from an external command does not have a trailing newline, Nushell will crash when trying to read it.

To Reproduce Bug was discovered using hub. I use this utility to query the Github API.

The following command crashes the shell:

❯ hub api 'orgs/nushell/repos' | from-json
thread 'main' panicked at 'Internal error: could not read lines of text from stdin', src/commands/classified/external.rs:276:22
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace.

I think this is caused by the lack of newline and the end of the output from hub. ZSH shows a % at the end of the output.

Note the following works and simply prints the raw JSON:

❯ hub api 'orgs/nushell/repos'
[{"id":186024298,"node_id":"MDEwOlJlcG9zaXRvcnkxODYwMjQyOTg=","name":"nushell","full_name":"nushell/nushell","private":false,...

Expected behavior I would expect Nushell to be able to handle input without trailing newline.

So, the following should work:

❯ hub api 'orgs/nushell/repos' | from-json

Screenshots N/A

Configuration (please complete the following information): Nu 0.9.0 (built from master a couple of days ago)

closed time in 11 days

kvrhdn

push eventjonathandturner/nushell

Jonathan Turner

commit sha b5d904beacca2a3a1018d5a3c8d075724679e648

clippy

view details

push time in 11 days

PR opened nushell/nushell

Force a \n at the end of a stdout stream

Fixes #1355

+43 -2

0 comment

2 changed files

pr created time in 11 days

create barnchjonathandturner/nushell

branch : force_stdout_newline

created branch time in 11 days

delete branch jonathandturner/nushell

delete branch : no_textview_autoview

delete time in 11 days

PR closed nushell/nushell

No textview autoview

This removes the automatic textview from autoview. It would use this in cases where the anchor told the file type. Unfortunately, in practice this would trip up usage as it would take time to colorize and sometimes it would colorize text you didn't want to colorize.

It's easy enough to open textview manually at the end of a pipeline or open a file in an editor. This seems like a good idea that didn't quite pan out.

+14 -29

1 comment

2 changed files

jonathandturner

pr closed time in 11 days

delete branch jonathandturner/nushell

delete branch : multiline_script

delete time in 12 days

push eventnushell/nushell

Jonathan Turner

commit sha 8ae8ebd1076ea1f37db92349ef8d84dfaabd0ada

Add support for multiline script files (#1386) * Add support for multiline script files * clippy

view details

push time in 12 days

push eventjonathandturner/nushell

Jonathan Turner

commit sha 388b592acefb060d4290fe591bbe70dcbf376ab7

clippy

view details

push time in 12 days

PR opened nushell/nushell

Add support for multiline script files
+56 -20

0 comment

6 changed files

pr created time in 12 days

create barnchjonathandturner/nushell

branch : multiline_script

created branch time in 12 days

pull request commentnushell/contributor-book

add table of contents

Oh nice!

sebastian-xyz

comment created time in 12 days

push eventnushell/contributor-book

Shaurya Shubham

commit sha 0a60ec9cfcf03d919fb9995144a194ea1034ffe3

Fix minor typo

view details

Jonathan Turner

commit sha 31bc7c97447171e041aec75724e02fbc2a7ba6ca

Merge pull request #12 from coolshaurya/patch-1 Fix minor typo in README

view details

push time in 12 days

pull request commentnushell/contributor-book

Fix minor typo in README

Thanks!

coolshaurya

comment created time in 12 days

push eventnushell/contributor-book

Sebastian Jung

commit sha 54ac64a2d3780b464722c50472597e85cdcc2ff2

add table of contents

view details

Jonathan Turner

commit sha bd59fe78aa6b36eb26fd049f812b9d09a180a61c

Merge pull request #13 from sebastian-xyz/table-of-contents add table of contents

view details

push time in 12 days

pull request commentnushell/book

docs: fix missing backtick in coming_from_bash.md

@IgorMinar the example here is the pipeline ls | where type == Dir. We're working on aliases to make it smaller :) but basically the structured output from ls flows through the where command.

IgorMinar

comment created time in 12 days

push eventnushell/nushell

Jonathan Turner

commit sha 473e9f9422d26979fd695d083588b74821e4055a

Tiny improvement to sys (#1385)

view details

push time in 12 days

PR opened nushell/nushell

Tiny improvement to sys
+1 -0

0 comment

1 changed file

pr created time in 12 days

push eventnushell/nushell

Corvus Corax

commit sha 96985aa6925577c499853359e5fd4543ea9e1b30

Fix invalid shorthand flag (#1384)

view details

push time in 12 days

more