Support .Args.XYZ instead of stencil.Arg "XYZ"

[deregtd]

Args are so common, for quality-of-life, just fill them into the base go-template vars under Args or Arg

[jaredallard]

Yeah, this makes sense to do. I think we probably need to move the JSON schema validation to before render time as well.

[jaredallard]

Hmmmm, I'm not sure this is a good idea. That would break the default logic, which would make dealing with arguments significantly more annoying. For example, stencil.Arg "a.b.c" would parse the JSON schema and set the default value for that type if it's not set at accessing time, while {{ a.b.c }} would just return nil. We could retain that behaviour by building that entire map based on the JSON schema, but that feels much less clean for little benefit.

I'll still consider this, but I'm not sure if it's worth it.

[deregtd]

Oh are there defaults supported by the schema? I'm not sure I've ever used those... I would expect that defaults are made through usage in templates, not the schema. (i.e. if a.b.c != nil)

[jaredallard]

Oh are there defaults supported by the schema? I'm not sure I've ever used those... I would expect that defaults are made through usage in templates, not the schema. (i.e. if a.b.c != nil)

Hmmm, that's fair. I guess most people would be more familiar with default from sprig, also.

[deregtd]

Yeah that's the system I used several places. Basically a COALESCE() operator.