Add `changed_charts` and `chart_version` outputs

[n-oden]

This should let downstream steps make conditional decisions based on what we found.

Also: fix a confusing double-negative test in main().

Signed-off-by: Nathan J. Mehl n@oden.io

[n-oden]

@cpanato ping? :)

[n-oden]

@davidkarlsen ping? :)

[cpanato]

I think this will require you to change the action.yml as well to make that explicitly the outputs

[n-oden]

@cpanato per the docs declaring outputs in action.yml is optional:

If you don't declare an output in your action metadata file, you can still set outputs and use them in a workflow.

That said if you'd prefer to have them explicitly declared I can definitely do that.

[n-oden]

@cpanato ping? :)

[n-oden]

@cpanato thanks for the review! I've implemented your changes.

[n-oden]

@cpanato ping? :)

[afernandez-01]

+1

[cpanato]

@n-oden testing is failing because in the CI we only install and do not run the full action, so the changed_charts.txt and the other file does not exist, we might want to add a safe guard to avoid that error

[n-oden]

@cpanato fixed; try again?

[n-oden]

@cpanato ping? :)

[n-oden]

@davidkarlsen @cpanato hey folks, any chance we can get this merged?

[cpanato]

@n-oden i will test this again

[n-oden]

@cpanato all done, take another look?

[calvinrzachman]

This is a great addition! Any chance we could cut a new release containing these changes?