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

AzureAD/microsoft-authentication-library-for-js 1050

Microsoft Authentication Library (MSAL) for JS

jonathandturner/AudioScope-ng2 32

AudioScope demo written in Angular 2 + TypeScript

jonathandturner/actress 4

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

pull request commentnushell/nushell

Added `math product` support

@gdhuper - how's the PR going? Feel free to reach out if you need anything.

gdhuper

comment created time in 5 hours

push eventnushell/nushell

Bailey Layzer

commit sha e1c5940b042683eca82b338158dac47d194c7b67

Add command "reduce" (#2292) * initial * fold working * tests and cleanup * change command to reduce, with fold flag * move complex example to tests * add --numbered flag

view details

push time in 5 hours

PR merged nushell/nushell

Add command "reduce"

Re: #2267

reduce takes a list or table and accumulates it into a single value using a block. This block must be a function $acc: T, $it: T -> $acc': T, as reduce customarily uses the first item as the initial accumulator value. --fold <any> allows the user to specify the initial acc value, which may be of any type (as long as the supplied block is $acc: T, $it: S -> $acc': T).

Couple of loose ends:

  • My example echo a,b 1,2 3,4 | split column , | headers | reduce -f 1.6 { = $acc * $(echo $it.a | str to-int) + $(echo $it.b | str to-int) } works for me (latest from main, current Manjaro) but doesn't work with cargo test (reduce receives a stream with a single UntaggedValue::Nothing rather than the Rows it usually would) - does anyone know why this is?
  • each has --numbered to provide $it.index - my instinct is to add this here as well.
+289 -5

1 comment

7 changed files

bailey-layzer

pr closed time in 5 hours

pull request commentnushell/nushell

Add command "reduce"

Looks good!

bailey-layzer

comment created time in 5 hours

push eventnushell/nushell

k-brk

commit sha c48c092125bb75d6244029a41dc89908ac04acbb

String funcs - Contains and IndexOf (#2298) * Contains and index of string functions * Clippy and fmt

view details

push time in 16 hours

PR merged nushell/nushell

String funcs - Contains and IndexOf

Another two functions for string ( #2092 ) - contains and indexof. Contains features optional -i flag for case insensitive match. IndexOf features optional range parameter when one can specify start and/or end index of string.

+513 -3

1 comment

5 changed files

k-brk

pr closed time in 16 hours

pull request commentnushell/nushell

String funcs - Contains and IndexOf

Looks good!

k-brk

comment created time in 16 hours

pull request commentAzure/azure-sdk-for-js

[Identity] Renamed the VSCodeCredential to VisualStudioCodeCredential

I don't see it in the code docs, but I did notice you'll need to build this, as the API review file wasn't update.

sadasant

comment created time in a day

PR closed Azure/azure-sdk-for-js

Reviewers
[Identity] Idea for accepting the AZURE_CLIENT_ID environment variable on the ManagedIdentityCredential Azure.Identity

This is just an idea. It's for this issue: https://github.com/Azure/azure-sdk-for-js/issues/5696

Feedback appreciated 🌞

+3 -0

1 comment

1 changed file

sadasant

pr closed time in a day

pull request commentAzure/azure-sdk-for-js

[Identity] Idea for accepting the AZURE_CLIENT_ID environment variable on the ManagedIdentityCredential

Just conferred with @schaabs. We decided not to do this, and instead read our settings from DAC only, for client Ids that are being inferred rather than explicitly passed in.

Going ahead and closing.

sadasant

comment created time in a day

pull request commentAzure/azure-sdk-for-js

[Identity] Renamed the VSCodeCredential to VisualStudioCodeCredential

Do we happen to talk about VSCodeCredential in any docs, by chance?

sadasant

comment created time in a day

Pull request review commentAzure/azure-sdk-for-js

[Identity] Concept PR for the token refresher update

 describe("BearerTokenAuthenticationPolicy", function() {     );   }); -  it("refreshes access tokens when they expire", async () => {+  it("refreshes access tokens when they expire", async function() {     const now = Date.now();     const refreshCred1 = new MockRefreshAzureCredential(now);     const refreshCred2 = new MockRefreshAzureCredential(now + TokenRefreshBufferMs);     const notRefreshCred1 = new MockRefreshAzureCredential(now + TokenRefreshBufferMs + 5000);      const credentialsToTest: [MockRefreshAzureCredential, number][] = [-      [refreshCred1, 2],-      [refreshCred2, 2],+      [refreshCred1, 1],+      [refreshCred2, 1],

Ah okay, got it.

sadasant

comment created time in a day

Pull request review commentAzure/azure-sdk-for-js

[Identity] Concept PR for the token refresher update

 export class BearerTokenAuthenticationPolicy extends BaseRequestPolicy {     private tokenCache: AccessTokenCache   ) {     super(nextPolicy, options);+    const requiredMillisecondsBeforeNewRefresh = 30000;

You're hardcoding 30000 here. What does it mean? Why 30000?

sadasant

comment created time in a day

issue commentnushell/nushell

Env var shorthand notation seems to only support a single env var

@Kelli314 - one possible direction would be to not update with-env, and instead wrap with new with-env for each variable.

Something like:

FOO=1 BAR=2 my command

Could expand into:

with-env [FOO 1] { with-env [BAR 2] { my command } }
thegedge

comment created time in a day

push eventnushell/nushell

Joseph T. Lyons

commit sha eeb9b4edcbdf65d975f4114318552e52e293c57a

Match cleanup (#2294) * Delete unnecessary match * Use `unwrap_or_else()` * Whitespace was trim on file save * Use `map_or_else()` * Use a default to group all match arms with same output * Clippy made me do it

view details

push time in a day

PR merged nushell/nushell

Match cleanup

Keeping these a bit smaller now. There's a lot to sift through still, but I'll probably take a break from this and move on to some other things, and come back to it later.

+8 -24

1 comment

5 changed files

JosephTLyons

pr closed time in a day

pull request commentnushell/nushell

Match cleanup

Looks good!

JosephTLyons

comment created time in a day

push eventnushell/nushell

Kornel

commit sha 3a7869b422c8938880e2de4e2d16fe712dc9aaad

Switch to maintained app_dirs (#2293) * Switch to maintained app_dirs * Update app_dirs under old name

view details

push time in a day

PR merged nushell/nushell

Switch to maintained app_dirs
+1 -1

1 comment

1 changed file

kornelski

pr closed time in a day

pull request commentnushell/nushell

Switch to maintained app_dirs

Thanks!

kornelski

comment created time in a day

pull request commentnushell/nushell

added pkg_mgrs/winget + yaml

We could use one of the test icons, I guess.

How hard would it be to sign the msi? Seems like something we could sign up for and then add it to CI.

fdncred

comment created time in a day

delete branch nushell/demo

delete branch : themeing-adjustments2

delete time in 2 days

push eventnushell/demo

Jörn Zaefferer

commit sha 72aabe5038a7c5e204e41ecaab67ee439c8d9305

use new --partial option make zebra row styling more obvious add some padding to non-table output (and override it for tables) improve layout for lists, too Fixes #45 Closes #52 Closes #53

view details

Jonathan Turner

commit sha 4c61fce3e71e90fdcda8c974b2d9692b4fd01459

Merge pull request #54 from nushell/themeing-adjustments2 use new --partial option

view details

push time in 2 days

PR merged nushell/demo

use new --partial option

make zebra row styling more obvious

add some padding to non-table output (and override it for tables)

improve layout for lists, too

Fixes #45 Closes #52 Closes #53

Preview: Screenshot 2020-08-03 at 08 36 11

+16 -1

1 comment

2 changed files

jzaefferer

pr closed time in 2 days

PR closed nushell/demo

adjust theme, more contrast between body and output

Fixes #45 Closes #52

Screenshot 2020-08-01 at 21 50 13 Screenshot 2020-08-01 at 21 50 18

+5 -1

0 comment

1 changed file

jzaefferer

pr closed time in 2 days

issue closednushell/demo

row colors a bit too light

Now that the alternating rows are a bit lighter, it's harder to see the difference between them. I think having at least some alternative color would be nice to make the tables more readable.

Here's what it looks like for me in Linux/Firefox

Screenshot from 2020-07-26 14-59-08

closed time in 2 days

jonathandturner

pull request commentnushell/demo

use new --partial option

Oh good! Thanks @jzaefferer

jzaefferer

comment created time in 2 days

Pull request review commentAzure/azure-sdk-for-js

[Identity] Concept PR for the token refresher update

 describe("BearerTokenAuthenticationPolicy", function() {     );   }); -  it("refreshes access tokens when they expire", async () => {+  it("refreshes access tokens when they expire", async function() {     const now = Date.now();     const refreshCred1 = new MockRefreshAzureCredential(now);     const refreshCred2 = new MockRefreshAzureCredential(now + TokenRefreshBufferMs);     const notRefreshCred1 = new MockRefreshAzureCredential(now + TokenRefreshBufferMs + 5000);      const credentialsToTest: [MockRefreshAzureCredential, number][] = [-      [refreshCred1, 2],-      [refreshCred2, 2],+      [refreshCred1, 1],+      [refreshCred2, 1],

What's the difference between 1 and 2 here?

sadasant

comment created time in 2 days

Pull request review commentAzure/azure-sdk-for-js

[Identity] Concept PR for the token refresher update

 export class BearerTokenAuthenticationPolicy extends BaseRequestPolicy {     private tokenCache: AccessTokenCache   ) {     super(nextPolicy, options);+    const requiredMillisecondsBeforeNewRefresh = 30000;

Commenting so we don't forget it - probably don't want to hardcode the delay like this. At the very least, we should pick the buffer, make it a const, and document it.

sadasant

comment created time in 2 days

issue commentAzure/azure-sdk-for-js

[FEATURE REQ] AzureCliCredential and all new credentials need to be added to docs

@jongio - going back over this now. Will try to track down why the docs are missing.

The first link should probably be: https://docs.microsoft.com/en-us/javascript/api/@azure/identity/defaultazurecredential?view=azure-node-preview

Since the docs are still in preview.

For the second link, the AzureCliCredentials and VSCodeCredentials are documented.

That said, what documentation is there is a bit on the slim side. We may want to add more front matter docs.

jongio

comment created time in 2 days

issue closednushell/demo

`ls` examples don't work in wasm

ls *.foo doesn't work in wasm. Doesn't look like we support any wildcards (yet?)

closed time in 2 days

jonathandturner

issue commentnushell/demo

`ls` examples don't work in wasm

Thanks for looking. I'll close for now, since you're right it feels like something that's outside of "just a demo"

jonathandturner

comment created time in 2 days

issue commentnushell/demo

config.toml location

Happy to help here. I'm not sure if we can set the config file location as we start up Nu, but it shouldn't be too bad to add it if it's not yet supported.

fdncred

comment created time in 2 days

issue closednushell/demo

syntax highlighting

Nu makes syntax highlighting available internally, so in theory we could expose this to wasm-nu, and then use the syntax highlighting information to color the user input.

So long as we don't interfere with them typing, it should be okay?

closed time in 2 days

jonathandturner

issue commentnushell/demo

syntax highlighting

Let's go ahead and close for now. We can get fancy when we feel like doing a "2.0"

jonathandturner

comment created time in 2 days

Pull request review commentnushell/nushell

Add command "reduce"

+use crate::commands::classified::block::run_block;+use crate::commands::WholeStreamCommand;+use crate::prelude::*;+use crate::{CommandArgs, CommandRegistry, Example, OutputStream};+use futures::stream::once;+use nu_errors::ShellError;+use nu_protocol::{hir::Block, Primitive, Scope, Signature, SyntaxShape, UntaggedValue, Value};++pub struct Reduce;++#[derive(Deserialize)]+pub struct ReduceArgs {+    block: Block,+    fold: Option<Value>,+}++#[async_trait]+impl WholeStreamCommand for Reduce {+    fn name(&self) -> &str {+        "reduce"+    }++    fn signature(&self) -> Signature {+        Signature::build("reduce")+            .named(+                "fold",+                SyntaxShape::Any,+                "reduce with initial value",+                Some('f'),+            )+            .required("block", SyntaxShape::Block, "reducing function")+    }++    fn usage(&self) -> &str {+        "Aggregate a list table to a single value using an accumulator block. Block must be+        (A, A) -> A unless --fold is selected, in which case it may be A, B -> A."+    }++    async fn run(+        &self,+        args: CommandArgs,+        registry: &CommandRegistry,+    ) -> Result<OutputStream, ShellError> {+        reduce(args, registry).await+    }++    fn examples(&self) -> Vec<Example> {+        vec![+            Example {+                description: "Simple summation (equivalent to math sum)",+                example: "echo 1 2 3 4 | reduce { = $acc + $it }",+                result: Some(vec![UntaggedValue::int(10).into()]),+            },+            Example {+                description: "Summation from starting value using fold",+                example: "echo 1 2 3 4 | reduce -f $(= -1) { = $acc + $it }",+                result: Some(vec![UntaggedValue::int(9).into()]),+            },+            Example {+                // TODO this appears to work, but not in testing+                description: "Folding with rows",+                example: r#"echo a,b 1,2 3,4 | split column , | headers | reduce -f 1.6 { = $acc * $(echo $it.a | str to-int) + $(echo $it.b | str to-int) }"#,

In examples, for testing purposes we only allow echo, math expressions, and the command being tested. You'll need to simplify this example if you want to test. Alternatively, you could use None on line 63 and turn off testing for this example.

bailey-layzer

comment created time in 2 days

delete branch jonathandturner/nushell

delete branch : partial_html

delete time in 2 days

push eventnushell/nushell

Jonathan Turner

commit sha f33da33626e9dca73b3d855013199f815101e287

Add --partial to 'to html' (#2291)

view details

push time in 2 days

PR merged nushell/nushell

Add --partial to 'to html'

This adds the ability to only output html-ized contents of the html, rather than a full page of html. With this, it's easier to use to html to create results that can be embedded in an existing webpage, like you would with jupyter/wasm.

I've also removed the coloring of the table, since the style for the body is set and should be sufficient (unless I missed a case where this doesn't work).

+55 -28

0 comment

2 changed files

jonathandturner

pr closed time in 2 days

push eventnushell/notes

Jonathan Turner

commit sha 0d3104c205ecb361cc74ecb0999fea48174a5a6f

Update 07-31-2020.md

view details

push time in 2 days

issue closedjonathandturner/rhai

Move Rhai to its own org/repo?

@schungx (and other contributors)

It's been a long while since I was actively involved with Rhai. It's been great to see it continue to bloom under your care.

I was wondering if we should consider finding a good home for this repo. While it's fine with me to continue to have it live here, I don't think it reflects the current maturity or authorship of Rhai today.

Thoughts?

closed time in 2 days

jonathandturner

issue commentjonathandturner/rhai

Move Rhai to its own org/repo?

Sounds good. Will close this issue for now.

jonathandturner

comment created time in 2 days

PR opened nushell/nushell

Add --partial to 'to html'

This adds the ability to only output html-ized contents of the html, rather than a full page of html. With this, it's easier to use to html to create results that can be embedded in an existing webpage, like you would with jupyter/wasm.

I've also removed the coloring of the table, since the style for the body is set and should be sufficient (unless I missed a case where this doesn't work).

+55 -28

0 comment

2 changed files

pr created time in 2 days

create barnchjonathandturner/nushell

branch : partial_html

created branch time in 2 days

PR closed nushell/nushell

Try adding no color to body in 'to html'

An experiment to see if adding the background color to a wrapping div works better than at the body level.

cc @jzaefferer / @fdncred

+7 -17

1 comment

1 changed file

jonathandturner

pr closed time in 2 days

pull request commentnushell/nushell

Try adding no color to body in 'to html'

Going to do this slightly differently. Closing this PR so I can create a new one.

jonathandturner

comment created time in 2 days

push eventnushell/book

Thomas B Homburg

commit sha 299c76e6225b7dbc0da4aab83bc095155aa922df

Add example of using command output as argument

view details

Thomas B Homburg

commit sha 4eed413aad0f689029ab2bf76b67e2b7d4ae42c8

Use better argument syntax

view details

Jonathan Turner

commit sha 184323f2f1fbd3828294143eacc3d059c406d6cb

Merge pull request #118 from homburg/patch-1 [doc] Add example of using command output as argument compared to bash

view details

push time in 2 days

pull request commentnushell/book

[doc] Add example of using command output as argument compared to bash

Thanks!

homburg

comment created time in 2 days

push eventnushell/nushell

Shaurya Shubham

commit sha a88f5c7ae725969b330a72a913aec24dcfbbbf80

Make `str collect` take an optional separator value (#2289) * Make `str collect` take an optional separator value * Make `str collect` take an optional separator value * Add some tests * Fix my tests

view details

push time in 3 days

issue closednushell/nushell

Have `str collect` take an optional separator value

Is your feature request related to a problem? Please describe. It would be nice to have str collect take a optional separator value

Describe the solution you'd like With this - echo 1..5 | str from | str collect "|" will return 1|2|3|4|5

echo a b c d e f g h | str from | str collect $(char newline) will return

a
b
c
d
e
f
g
h

Describe alternatives you've considered Make a new subcommand in str for this functionality

Additional context None

closed time in 3 days

coolshaurya

pull request commentnushell/nushell

Make `str collect` take an optional separator value

Looks good!

coolshaurya

comment created time in 3 days

pull request commentnushell/nushell

Make `str collect` take an optional separator value

@coolshaurya looks like the tests aren't building. Are you able to run the tests locally?

coolshaurya

comment created time in 3 days

PR opened nushell/nushell

Try adding no color to body in 'to html'

An experiment to see if adding the background color to a wrapping div works better than at the body level.

cc @jzaefferer / @fdncred

+7 -17

0 comment

1 changed file

pr created time in 3 days

create barnchjonathandturner/nushell

branch : remove_body_html

created branch time in 3 days

push eventjonathandturner/nushell

Jörn Zaefferer

commit sha 4ef15b5f8020a3f51d346d24c245d2956d7e3bc9

docs/alias: simplify the 'persistent' section, using --save (#2285) All the workarounds using `config` aren't necessary anymore. Only `config path` is still of interest.

view details

Joseph T. Lyons

commit sha 9fb6f5cd090533e0c9472cdd93c6993a9c93eb75

Change `f`/`full` flag to `l`/`long` for `ls` and `ps` commands (#2283) * Change `f`/`full` flag to `l`/`long` for `ls` and `ps` commands * Fix a few more `--full` instances

view details

Joseph T. Lyons

commit sha ee734873ba3eec05b239bf8b55f77351768cb015

Fix no longer working histogram example (#2271) * Fix no longer working histogram example * Oops

view details

Jason Gedge

commit sha cda53b6cdaf81633db424f3e55fbcf4b821d9642

Return incomplete parse from lite_parse (#2284) * Move lite_parse tests into a submodule * Have lite_parse return partial parses when error encountered. Although a parse fails, we can generally still return what was successfully parsed. This is useful, for example, when figuring out completions at some cursor position, because we can map the cursor to something more structured (e.g., cursor is at a flag name).

view details

Jonathan turner

commit sha 968dbcba15e55062d8ad0dae029504504261ff7c

merge main

view details

push time in 3 days

create barnchjonathandturner/nushell

branch : try_no_body_html

created branch time in 3 days

issue commentnushell/nushell

Up- and Down- Arrow should consider entered text

@oberien - we currently use https://github.com/kkawakam/rustyline to do our line editing. If it gets implemented there, we should be able to add it nushell. You might want to file an issue on that repo and see if it's being worked on. The bonus is that other people using rustyline would also benefit.

Flare576

comment created time in 3 days

push eventnushell/nushell

Jason Gedge

commit sha cda53b6cdaf81633db424f3e55fbcf4b821d9642

Return incomplete parse from lite_parse (#2284) * Move lite_parse tests into a submodule * Have lite_parse return partial parses when error encountered. Although a parse fails, we can generally still return what was successfully parsed. This is useful, for example, when figuring out completions at some cursor position, because we can map the cursor to something more structured (e.g., cursor is at a flag name).

view details

push time in 3 days

PR merged nushell/nushell

Return incomplete parse from lite_parse

What?

Whenever a parse failure is encountered in lite_parse, instead of just returning a parse error, we return a struct that has both the cause of the parse error, and optionally what was successfully parsed up to that point. For example:

command arg1 "arg2 --test

would fail to parse previously, because the quoted string is unterminated. We now return the same error, but we also return a command with two args: arg1 and "arg2 --test".

Why?

As part of attempts to improve completions by making use of signatures and the parser to understand the context of what we need to complete (see https://github.com/nushell/rfcs/pull/2), we need partial parsing. Completions will often be requested with an incomplete line, so parsing will typically fail. We can still make use of what was successfully parsed to help guide the completion engine.

+308 -230

0 comment

4 changed files

thegedge

pr closed time in 3 days

push eventnushell/nushell

Joseph T. Lyons

commit sha ee734873ba3eec05b239bf8b55f77351768cb015

Fix no longer working histogram example (#2271) * Fix no longer working histogram example * Oops

view details

push time in 3 days

PR merged nushell/nushell

Fix no longer working histogram example

https://github.com/nushell/nushell/issues/2226

+1 -1

3 comments

1 changed file

JosephTLyons

pr closed time in 3 days

push eventnushell/nushell

Joseph T. Lyons

commit sha 9fb6f5cd090533e0c9472cdd93c6993a9c93eb75

Change `f`/`full` flag to `l`/`long` for `ls` and `ps` commands (#2283) * Change `f`/`full` flag to `l`/`long` for `ls` and `ps` commands * Fix a few more `--full` instances

view details

push time in 3 days

issue commentnushell/demo

row colors a bit too light

I think by default to html shouldn't add colors, though I think we should make it easy for people to pass in a flag to configure the CSS that will be used. The theming @fdncred could be possible with a flag like --theme=theme_name.

jonathandturner

comment created time in 3 days

issue commentnushell/demo

unhandled panics are mostly swallowed

Ah, that could be. Adding a simple fix to see if that's what it is.

jzaefferer

comment created time in 3 days

push eventnushell/demo

Jonathan Turner

commit sha 84781bc77c025d95c1dbb37c342ec46820cf7b9d

Update index.js

view details

Jonathan Turner

commit sha b4a553ce0423b4a3294ead97c2e481ae523bf673

Merge pull request #51 from nushell/try_catch_wasm_nu Add try-catch for running wasm

view details

push time in 3 days

delete branch nushell/demo

delete branch : try_catch_wasm_nu

delete time in 3 days

PR merged nushell/demo

Add try-catch for running wasm

Let's see if this helps with the panics

+6 -1

1 comment

1 changed file

jonathandturner

pr closed time in 3 days

pull request commentnushell/demo

Add try-catch for running wasm

Let's see if this improves https://github.com/nushell/demo/issues/12

jonathandturner

comment created time in 3 days

PR opened nushell/demo

Add try-catch for running wasm

Let's see if this helps with the panics

+6 -1

0 comment

1 changed file

pr created time in 3 days

create barnchnushell/demo

branch : try_catch_wasm_nu

created branch time in 3 days

delete branch nushell/demo

delete branch : custom-file

delete time in 3 days

push eventnushell/demo

Jörn Zaefferer

commit sha 1d27edf27d47747bcf338f15b54570c422834f88

provide a textarea for pasting a custom file, hidden in a detail element For that, move the LocalStorage fs into the MountableFileSystem, so it can exist in parallel to the ZipFS one. Move the samples to /samples and add a shortcut command to list those files. Modify the command-click handler to match all clicks with data-command attributes, not just inside #examples. Fixes the bug where clicking around the sample buttons would run an empty-string command. Since there's now more commands, optimize some labels and the layout, also making it wrap on smaller screens

view details

Jonathan Turner

commit sha 88aafb566a323e55d1a3c950933d9490f9ac5d86

Merge pull request #35 from nushell/custom-file provide a textarea for pasting a custom file

view details

push time in 3 days

PR merged nushell/demo

provide a textarea for pasting a custom file

currently the ZipFS conflicts with this; with only the LocalStorage fs it works

Fixes #9

+33 -22

4 comments

3 changed files

jzaefferer

pr closed time in 3 days

issue closednushell/demo

As editor for json and other formats

To make this more than a slightly crippled demo for nushell, we could offer a textarea (and upload/dnd section) for providing JSON, toml, csv or other data that nushell can parse (maybe auto detect the format), then a pipeline for processing it and an output area for the result, as HTML table and whatever structured output is desired (default to input format, can switch to something else.

The format selects could be fed with a list of subcommands from "from" and "to".

Since everything runs on the client, we could update everything automatically while typing, as long as the pipeline is a valid command

closed time in 3 days

jzaefferer

push eventnushell/demo

Jörn Zaefferer

commit sha 2916eaee1ab85f36ceb48159ed7e8912e601e698

Use demo output for super simple loading indicator Just a cursive text message, but since that gets replaced as soon as everything is ready, it works fine. Fixes #43

view details

Jörn Zaefferer

commit sha 1768cc82a36f13b804900e1dd03273d8348010ca

add a subtle pulse animation to the loading indicator text

view details

Jonathan Turner

commit sha 746df301dbb855f60ad85c269991a7dc4af26647

Merge branch 'main' into loading-indicator

view details

Jonathan Turner

commit sha e57181f1450d05a16e7dcbf96a6b31d9ad1c5b06

Merge pull request #50 from nushell/loading-indicator Use demo output for super simple loading indicator

view details

push time in 3 days

delete branch nushell/demo

delete branch : loading-indicator

delete time in 3 days

PR merged nushell/demo

Use demo output for super simple loading indicator

Just a cursive text message, but since that gets replaced as soon as everything is ready, it works fine.

Fixes #43

Preview: Screenshot 2020-08-01 at 13 31 36

+14 -2

1 comment

2 changed files

jzaefferer

pr closed time in 3 days

issue closednushell/demo

Loading indicator

Loading the several MB of wasm and the zip file can take a while, and until then the demo shows up, but isn't usable. A simple loading indicator would help

Maybe via https://codepen.io/dsenneff/pen/jOWdvqx

closed time in 3 days

jzaefferer

push eventnushell/demo

Jörn Zaefferer

commit sha a6fc5390459a5a23b0bf81a077855b6a40329d79

Add GitHub ribbon and WIP note at the bottom ribbon in the top right, based on https://gist.github.com/thunsaker/1188641/61939b58f5e37922602d8e7dd65696e6628b5b59 but with a green background and in the top-right A link to the "WIP" issue with a note at the bottom Fixes #41 Fixes #46

view details

Jörn Zaefferer

commit sha cc0caa5967986bfcfb9efb5994b0ea77a489da73

add button to pivot overflowing tables Also uses `rename` to give the (likely) two columns nicer names. Fixes #29

view details

Jonathan Turner

commit sha f9965884c6da62fed7364f9ffb980e242903d34d

Merge pull request #48 from nushell/github-links Add GitHub ribbon and WIP note at the bottom

view details

Jonathan Turner

commit sha 3fe9b1f5623f6a4eba48e4681593e57b5108df54

Merge pull request #49 from nushell/pivot-hint add button to pivot overflowing tables

view details

Jonathan Turner

commit sha 746df301dbb855f60ad85c269991a7dc4af26647

Merge branch 'main' into loading-indicator

view details

push time in 3 days

push eventnushell/demo

Jörn Zaefferer

commit sha cc0caa5967986bfcfb9efb5994b0ea77a489da73

add button to pivot overflowing tables Also uses `rename` to give the (likely) two columns nicer names. Fixes #29

view details

Jonathan Turner

commit sha 3fe9b1f5623f6a4eba48e4681593e57b5108df54

Merge pull request #49 from nushell/pivot-hint add button to pivot overflowing tables

view details

push time in 3 days

delete branch nushell/demo

delete branch : pivot-hint

delete time in 3 days

PR merged nushell/demo

add button to pivot overflowing tables

Also uses rename to give the (likely) two columns nicer names.

Fixes #29

Table with overflow: Screenshot 2020-08-01 at 13 11 05

After running with the extra | pivot | rename key value: Screenshot 2020-08-01 at 13 11 14

+20 -2

5 comments

1 changed file

jzaefferer

pr closed time in 3 days

issue closednushell/demo

Should we add auto-rotation?

We use auto-rotation in the terminal to help fit things on the screen when it's a single long row. Feels like we might want to consider that for wasm-nu also:

Screenshot from 2020-07-20 06-55-50

Notice size of scrollbar.

closed time in 3 days

jonathandturner

pull request commentnushell/demo

add button to pivot overflowing tables

Cool, I like it 👍

jzaefferer

comment created time in 3 days

delete branch nushell/demo

delete branch : github-links

delete time in 3 days

push eventnushell/demo

Jörn Zaefferer

commit sha a6fc5390459a5a23b0bf81a077855b6a40329d79

Add GitHub ribbon and WIP note at the bottom ribbon in the top right, based on https://gist.github.com/thunsaker/1188641/61939b58f5e37922602d8e7dd65696e6628b5b59 but with a green background and in the top-right A link to the "WIP" issue with a note at the bottom Fixes #41 Fixes #46

view details

Jonathan Turner

commit sha f9965884c6da62fed7364f9ffb980e242903d34d

Merge pull request #48 from nushell/github-links Add GitHub ribbon and WIP note at the bottom

view details

push time in 3 days

PR merged nushell/demo

Add GitHub ribbon and WIP note at the bottom

ribbon in the top right, based on https://gist.github.com/thunsaker/1188641/61939b58f5e37922602d8e7dd65696e6628b5b59 but with a green background and in the top-right

A link to the "WIP" issue with a note at the bottom

Fixes #41 Fixes #46

Preview:

Screenshot 2020-08-01 at 12 35 52

+34 -0

0 comment

1 changed file

jzaefferer

pr closed time in 3 days

issue closednushell/demo

Add a link to this repo

Maybe using the widespread octocat banner in the top right

closed time in 3 days

jzaefferer

issue closednushell/demo

Readme button?

Was thinking that maybe there should be a readme button somewhere on the page that tells what's working and what's not working (i.e. @jzaefferer 's issue that lists it). Error handling is still in it's infancy and I don't want people trying it and getting aggravated and then assuming nushell as a whole doesn't support all the commands we claim it does.

closed time in 3 days

fdncred
MemberEvent
MemberEvent

push eventnushell/nushell.github.io

Jörn Zaefferer

commit sha 5833ae629a2e4bba78a38b7c1dc38cd4cd827174

navigation: add link to demo, fix link to blog The old blog link takes a while to redirect, a direct link is better. Also use https links, not http. The Let's Encrypt certs are fine.

view details

Jonathan Turner

commit sha de5ae97d11626b7ef4b3e23efdf5c7612320227b

Merge pull request #57 from jzaefferer/add-demo-link navigation: add link to demo, fix link to blog

view details

push time in 3 days

PR merged nushell/nushell.github.io

navigation: add link to demo, fix link to blog

The old blog link takes a while to redirect, a direct link is better. Also use https links, not http. The Let's Encrypt certs are fine.

+2 -1

0 comment

1 changed file

jzaefferer

pr closed time in 3 days

push eventnushell/nushell.github.io

Jörn Zaefferer

commit sha 833ad1115416ad73d8b4c3baac60de8ef8da99ca

copy and update CONTRIBUTING.md from the book repo Updated jekyll from 3.8 to 4.0, to match the bundler version in use here (2.1.4; 3.8 ships with 2.0.2) Removed the matching, Docker-less instructions from the README.

view details

Jonathan Turner

commit sha 4b882e7f6c61f2f192c4c46192b56271c5de27fb

Merge pull request #56 from jzaefferer/contributing-with-docker copy and update CONTRIBUTING.md from the book repo

view details

push time in 3 days

more