Deno section

[vladimyr]

Description

Show Deno version if the folder contains conventional Deno-specific files.

Screenshot

[Runrioter]

Now deno is not GA, so we may wait a while.

Early deno version:

➜ deno --version
deno: 0.2.8
v8: 7.2.502.16
typescript: 3.2.1

Now deno version:

➜ deno --version
deno 1.0.0-rc2
v8 8.4.300
typescript 3.8.3

Note the missing : char

Use deno -V print the only deno version

➜ deno -V
deno 1.0.0-rc2

Personally, I don't like the unofficial dvm. Just deno binary is enough.

BTW, please test your code.

[vladimyr]

Now deno is not GA, so we may wait a while.

Sure, I wholeheartedly agree and this is just a preparation step for possible future demand.

Note the missing : char

Why is it important, the delimiter is kept the same?

Use deno -V print the only deno version

Nice, thank you! Do you happen to know was it there from the beginning or added later on? 🤔

Personally, I don't like the unofficial dvm. Just deno binary is enough.

All vm -s are unofficial, aren't they? This is done in the same vein as the node section which does include nvm support.

BTW, please test your code.

I forgot to test it without dvm, my bad 🙃

Thanks for the quick review! 👍

[vladimyr]

Use deno -V print the only deno version

Nice, thank you! Do you happen to know was it there from the beginning or added later on? 🤔

The -V flag (short version) has introduced in v0.25.0 thus the only failsafe way to get deno version is:

$ deno --version | head -n1 | cut -d' ' -f2

🤔

[denysdovhan]

Thanks for contributing!

The code looks good. Please add tests for this section.

[binyamin]

• Deno has a config file, deno.json or deno.jsonc. The section should be shown if that file exists.
• Why does this code need to check dvm? It doesn't really matter how deno gets there. According to justjavac/dvm#83, the deno command would resolve to a dvm installation if it exists, and a system installation if it doesn't.
• Deno CLI is now at v1.25.3, so I think it's safe to use deno -V instead of deno --version, and make use of the shorter output.

[denysdovhan]

Merged in c43f8c5

[denysdovhan]

Thanks for contributing! Authorship is preserved.