rsconstruct 0.9.122

Rust based fast build system
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

//! The `env-var` rule: environment variable names.

use super::{Rule, RuleBase};
use crate::engines::actionlint::ast::{Env, Job, Step, Workflow};
use crate::engines::actionlint::yaml::go_quote;

pub struct RuleEnvVar {
    base: RuleBase,
}

impl RuleEnvVar {
    pub const fn new() -> Self {
        Self {
            base: RuleBase::new("env-var"),
        }
    }

    fn check_env(&mut self, env: Option<&Env>) {
        let Some(env) = env else {
            return;
        };
        if env.expression.is_some() {
            return;
        }
        for v in env.vars.values() {
            if v.name.contains_expression() {
                continue;
            }
            if v.name.value.contains(['&', '=', ' ', '\t']) {
                self.base.error(
                    v.name.pos,
                    format!(
                        "environment variable name {} is invalid. '&', '=' and spaces should not be contained",
                        go_quote(&v.name.value)
                    ),
                );
            }
        }
    }
}

impl Rule for RuleEnvVar {
    fn base(&mut self) -> &mut RuleBase {
        &mut self.base
    }

    fn visit_step(&mut self, step: &Step) {
        self.check_env(step.env.as_ref());
    }

    fn visit_job_pre(&mut self, job: &Job) {
        self.check_env(job.env.as_ref());
        if let Some(c) = &job.container {
            self.check_env(c.env.as_ref());
        }
        if let Some(s) = &job.services {
            for service in s.value.values() {
                self.check_env(service.container.env.as_ref());
            }
        }
    }

    fn visit_workflow_pre(&mut self, workflow: &Workflow) {
        self.check_env(workflow.env.as_ref());
    }
}