ansible / galaxy_collection

Collection of modules and roles to configure Automation Hub

GNU General Public License v3.0

54 stars 50 forks source link

README docfix #391

Closed syntaxsdev closed 1 month ago

syntaxsdev commented 1 month ago

fixed table readme for ee-image

What does this PR do?

Fixed the table for the README.md for the role ee-image

How should this be tested?

No test needed

Is there a relevant Issue open for this?

There is no relevant issue or PR for this

Other Relevant info, PRs, etc

N/A

syntaxsdev commented 1 month ago

PR just had Readme fixes, ready to merge

sean-m-sullivan commented 1 month ago

Doing a review on this, the new name was likely put in docs to be hopeful, and I believe we aren't able to rename image names through the API, the module does not support it, ref here

Additionally, the present we have set to be quoted so it shows up as a variable, as referenced below here here

We welcome the contributions, and feedback to improve the docs, I do not doubt we have some inconsistencies and typos around that we've missed, I hope you find more!

syntaxsdev commented 1 month ago

Doing a review on this, the new name was likely put in docs to be hopeful, and I believe we aren't able to rename image names through the API, the module does not support it, ref here

Additionally, the present we have set to be quoted so it shows up as a variable, as referenced below here here

We welcome the contributions, and feedback to improve the docs, I do not doubt we have some inconsistencies and typos around that we've missed, I hope you find more!

Thanks, I have corrected the suggestions.