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

Supply Alibaba Cloud Redis #88

Merged
merged 6 commits into from
Jul 1, 2021

Conversation

zzxwill
Copy link
Collaborator

@zzxwill zzxwill commented Jun 30, 2021

Description of your changes

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

xiqiu618 and others added 5 commits June 30, 2021 20:25
Signed-off-by: anhongbin <55675639+anhongbin@users.noreply.github.com>
Signed-off-by: Zheng Xi Zhou <zzxwill@gmail.com>
Co-authored-by: Zheng Xi Zhou <zzxwill@gmail.com>
Signed-off-by: anhongbin <55675639+anhongbin@users.noreply.github.com>
Signed-off-by: Zheng Xi Zhou <zzxwill@gmail.com>
Co-authored-by: Zheng Xi Zhou <zzxwill@gmail.com>
Signed-off-by: anhongbin <55675639+anhongbin@users.noreply.github.com>
Signed-off-by: Zheng Xi Zhou <zzxwill@gmail.com>
Co-authored-by: Zheng Xi Zhou <zzxwill@gmail.com>
Signed-off-by: anhongbin <55675639+anhongbin@users.noreply.github.com>
Signed-off-by: Zheng Xi Zhou <zzxwill@gmail.com>
Co-authored-by: Zheng Xi Zhou <zzxwill@gmail.com>
Signed-off-by: anhongbin <55675639+anhongbin@users.noreply.github.com>
Signed-off-by: Zheng Xi Zhou <zzxwill@gmail.com>
Signed-off-by: Zheng Xi Zhou <zzxwill@gmail.com>
@zzxwill zzxwill mentioned this pull request Jul 1, 2021
2 tasks
@zzxwill
Copy link
Collaborator Author

zzxwill commented Jul 1, 2021

For detailed reviews, please reference #68. In this PR, I just took it over to change tests to table-driven style and add some more tests to meet the coverage.

Copy link
Contributor

@wonderflow wonderflow left a comment

Choose a reason for hiding this comment

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

LGTM, thanks

@wonderflow wonderflow merged commit e9c9c12 into crossplane-contrib:master Jul 1, 2021
@zzxwill zzxwill deleted the supply_alibaba_redis branch July 1, 2021 14:29
@zzxwill zzxwill mentioned this pull request Jul 20, 2021
6 tasks
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

3 participants