nu-lint 1.2.0

Linter for Nu shell scripts that helpfully suggests improvements
Documentation 
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70

use super::RULE;

#[test]
fn ignores_external_without_hat() {
    RULE.assert_ignores("ddcutil setvcp 10");
}

#[test]
fn ignores_builtin_commands() {
    RULE.assert_ignores("ls");
}

#[test]
fn ignores_builtin_with_args() {
    RULE.assert_ignores("ls -la");
}

#[test]
fn ignores_pipeline_without_hat() {
    RULE.assert_ignores("git status | lines");
}

#[test]
fn ignores_function_definition() {
    RULE.assert_ignores(
        r#"
def my-command [] {
    echo "hello"
}
"#,
    );
}

#[test]
fn ignores_variable_assignment() {
    RULE.assert_ignores("let x = 10");
}

#[test]
fn ignores_if_statement() {
    RULE.assert_ignores("if true { echo yes }");
}

#[test]
fn ignores_for_loop() {
    RULE.assert_ignores("for i in 1..3 { echo $i }");
}

#[test]
fn ignores_hat_with_variable_args() {
    RULE.assert_ignores("let name = 'test'; ^echo $name");
}

#[test]
fn ignores_hat_in_pipeline() {
    RULE.assert_ignores("^ls | lines");
}

#[test]
fn ignores_hat_with_variable_command() {
    // When the command is a variable, the hat is necessary because the variable
    // might resolve to a builtin name at runtime
    RULE.assert_ignores("let file = 'test.txt'; ^$env.EDITOR $file");
}

#[test]
fn ignores_hat_with_env_variable_command() {
    // Environment variable as command - hat is required for safety
    RULE.assert_ignores("^$env.SHELL --version");
}