Skip to content

Conversation

@tsaoyu
Copy link

@tsaoyu tsaoyu commented Aug 11, 2022

Add Huawei cloud diagrams

disclaimer: although I am an employee of Huawei cloud, this project is not an official Huawei cloud product && is subject to changes

@mingrammer mingrammer added kind/feat/provider Provider request status/need-to-review Need to review labels Nov 9, 2022
Copy link
Collaborator

@gabriel-tessier gabriel-tessier left a comment

Choose a reason for hiding this comment

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

Thank your for the contribution, here small changes that need to be done before merging.

Please put your provider at the end of the files as reported. It will help when merging to resolve conflicts.
Also if you can run again the autogen script the resources are not in the website directory.

Sorry for the long delay.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you put the provider at the bottom of the lists/dicts, not in the middle.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you put the cleaner method at the end, and define it at the end.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/feat/provider Provider request status/need-to-review Need to review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants