ManageIQ / manageiq-content

Default ManageIQ content
Apache License 2.0
13 stars 120 forks source link

Fix powervs content files not matching YAML dump #707

Closed agrare closed 2 years ago

agrare commented 2 years ago

Oh just noticed that @jaywcarman already put a PR in for this https://github.com/ManageIQ/manageiq-content/pull/705

miq-bot commented 2 years ago

Checked commit https://github.com/agrare/manageiq-content/commit/c063531d75908a3c81379c40fda9e364db77630d with ruby 2.6.7, rubocop 1.19.1, haml-lint 0.35.0, and yamllint 0 files checked, 0 offenses detected Everything looks fine. :+1:

jaywcarman commented 2 years ago

Yeah, the diff on #705 is +18,939 −18,939 😱 I never got checks to pass though... (looks like you had to add trailing whitespace?).

agrare commented 2 years ago

I just rewrote the YAML files with File.writeYAML.load_file().to_yaml):

?> Dir.glob("content/automate/ManageIQ/System/Event/EmsEvent/IBMCloud-PowerVS.cl
ass/*.yaml").each do |f|
?>   File.write(f, YAML.load(File.read(f)).to_yaml)
>> end