cloudify-cosmo / repex

sed on steroids
Apache License 2.0
10 stars 3 forks source link

By default, repex should fail is there are unexpanded variables #67

Closed nir0s closed 7 years ago

nir0s commented 7 years ago

Currently, if no variables are provided but a variable placeholder exists, the placeholder will be taken as the with string. This creates a situation in which if you didn't provide the relevant variables, the replacement process will take place and you'll just see variable placeholders in your files.

We should, instead, fail if there are placeholders left in the config after variable expansion. We might also want to provide an --ignore-unexpanded-variables for whomever wishes to ignore them.