-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Removing the helm
and cloudFormation
sidecar deployment types
#452
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@salgattcy changes are looking good, could you please rebase this branch on top of main and re-generate the docs to make sure they are all up to date? I believe we need to update the docs for the sidecar_cft_template
data source. Thanks for working on this fix btw.
@VictorGFM , I don't see any conflicts. Why rebase? |
@wcmjunior In this PR we changed how metadata (the provider name to be more specific) is generated on the resource/data source docs, this was made to avoid generating unnecessary changes when running the |
Let's see if I did this all correctly 😬 |
Still doesn't make sense. There are no conflicts and the docs should look the same after being regenerated. |
48cb1e9
to
4ef9210
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Description of the change
Type of change
Checklists
Development
Code review
Testing