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

feat: Add tencentcloud provider #349

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

zqfan
Copy link
Contributor

@zqfan zqfan commented Oct 10, 2020

also add a basic provider unit test just to check autogen files

@zqfan
Copy link
Contributor Author

zqfan commented Oct 24, 2020

@mingrammer hello, is this pr available to merge, or should I do some modification to improve it?

@mingrammer mingrammer added kind/feat/provider Provider request status/need-to-review Need to review labels Oct 24, 2020
@mingrammer
Copy link
Owner

Sorry for late. I'll check it tomorrow!

diagrams/tencentcloud/container.py Show resolved Hide resolved
autogen.sh Outdated Show resolved Hide resolved
config.py Outdated Show resolved Hide resolved
@zqfan
Copy link
Contributor Author

zqfan commented Jan 31, 2021

@mingrammer config.py and authgen.sh have been fixed, and rebase on master latest code

@mingrammer
Copy link
Owner

Please resolve the conflicts.

@mingrammer mingrammer mentioned this pull request Nov 4, 2022
zqfan added 2 commits November 7, 2022 10:45
also add a basic provider unit test just to check autogen files
@zqfan
Copy link
Contributor Author

zqfan commented Nov 7, 2022

Please resolve the conflicts.

done

@mingrammer
Copy link
Owner

@zqfan Could you please add the recently updated services? For example, there are Elastic Kubernetes Service and Tencent Container Registry in the Container section: https://www.tencentcloud.com/ko/product.

Please check other sections too.

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.

2 participants