Remove undefined variable access

[dominikschulz]

This PR tries to work around some issues with strict_variables = true

[carlossg]

lint is failing

schedule was added in #46 mentioning that it shouldn't be in the parameters

[dominikschulz]

I'll fix the lint check and check on the metaparams for the future parser. But this parameter didn't work for us with the future parser enabled.

[carlossg]

ping @robbat2

[BobVincentatNCRdotcom]

I think the problem is that all branches of a selector are evaluated. I've submitted another patch that converts the selector to an if statement, and it passes tests.

[robbat2]

If you don't pass schedule down to the resource, the resource is always applied, which is not desired.

Has the lint changed, that metaparams must now be declared as well?

[BobVincentatNCRdotcom]

See #64 for an approach that passes tests.

[robbat2]

+1 on #64

[BobVincentatNCRdotcom]

Better approach in #66

[carlossg]

closing for #66