Skip to content
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

fix(acm): correct description for dvo #1915

Merged
merged 1 commit into from
Oct 19, 2023

Conversation

rushrs
Copy link
Contributor

@rushrs rushrs commented Oct 16, 2023

Description of your changes

Just fixing a spelling mistake that was bothering me whilst I was looking through the provider docs.

Fixes #

I have:

  • Read and followed Crossplane's contribution process.
  • Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

@rushrs rushrs changed the title Fix typo for acm aws provider fix(acm) typo for acm aws provider Oct 16, 2023
@rushrs rushrs force-pushed the patch-1 branch 2 times, most recently from f6185ae to 158ec93 Compare October 16, 2023 10:00
@MisterMX
Copy link
Collaborator

Thanks for the fixes, @rushrs. Could you please squash your changes into a single commit and append : to the conventional commit message, so it reads fix(acm):?

@rushrs
Copy link
Contributor Author

rushrs commented Oct 16, 2023

Thanks for the fixes, @rushrs. Could you please squash your changes into a single commit and append : to the conventional commit message, so it reads fix(acm):?

will do that now

@rushrs
Copy link
Contributor Author

rushrs commented Oct 16, 2023

just updated it @MisterMX

Signed-off-by: rushrs <17338080+rushrs@users.noreply.github.com>

update description for dvo

Signed-off-by: rushrs <17338080+rushrs@users.noreply.github.com>

update description for dvo

Signed-off-by: rushrs <17338080+rushrs@users.noreply.github.com>

update description for dvo api

Signed-off-by: rushrs <17338080+rushrs@users.noreply.github.com>

Update certificate_types.go

Signed-off-by: rushrs <17338080+rushrs@users.noreply.github.com>

another small formatting fix

Signed-off-by: rushrs <17338080+rushrs@users.noreply.github.com>

another small formatting fix

Signed-off-by: rushrs <17338080+rushrs@users.noreply.github.com>

add modified file

Signed-off-by: rushrs <17338080+rushrs@users.noreply.github.com>
@rushrs
Copy link
Contributor Author

rushrs commented Oct 18, 2023

Hi @MisterMX, I didn't realise this wasn't successful. i've made the changes to fix the error shown in CI now.

@MisterMX MisterMX changed the title fix(acm) typo for acm aws provider fix(acm): correct description for dvo Oct 19, 2023
Copy link
Collaborator

@MisterMX MisterMX left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thank you very much @rushrs!

@MisterMX MisterMX merged commit 9a50654 into crossplane-contrib:master Oct 19, 2023
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants