Add skip-validation flag; Remove default save path

[dhaiducek]

Feature:

• Adds a --skip-validation flag in order to do things like (you know, just off the top of my head 😂):

  str='{{ toYAML (set (fromYAML (fromConfigMap "default" "example" "file.yml")) "greeting" "hello") }}'
  echo "test: '${str}'" | template-resolver --skip-validation
  echo "test: |
    ${str}" | template-resolver --skip-validation

Fix:

• Removes the default s from save resources so that resources are not saved by default.

[yiraeChristineKim]

Do we need a unit test like your example

str='{{ toYAML (set (fromYAML (fromConfigMap "default" "example" "file.yml")) "greeting" "hello") }}'
echo "test: '${str}'" | template-resolver --skip-validation
echo "test: |
  ${str}" | template-resolver --skip-validation

[dhaiducek]

Do we need a unit test like your example

Yes, we most definitely do 🙂 I think I wasn't 100% sure this PR was desirable. I'll add a test, though!

[JustinKuli]

Does this only skip "validation" around Policy/ConfigurationPolicy details? As in, if I gave it an invalid template that doesn't close a {{ or something, that would still give an error of some kind, right?

If so, I'm wondering if this option should be renamed to "non-policy" or something similar.

[dhaiducek]

Does this only skip "validation" around Policy/ConfigurationPolicy details? As in, if I gave it an invalid template that doesn't close a {{ or something, that would still give an error of some kind, right?

If so, I'm wondering if this option should be renamed to "non-policy" or something similar.

That's correct. True, something like --skip-policy-validation?

[sonarqubecloud]

Quality Gate passed

Issues
4 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dhaiducek, JustinKuli

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [JustinKuli,dhaiducek]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment