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

Change func name CorePostMcisVm into CreateMcisVm #1219

Merged
merged 4 commits into from
Nov 1, 2022

Conversation

Rohit-R2000
Copy link
Contributor

changed function name from CorePostMcisVm to CreateMcisVm

changed function name from CorePostMcisVm to CreateMcisVm
@seokho-son
Copy link
Member

Hi @Rohit-R2000
Thank you for the contribution.

You need to search the text of the legacy func name in all files in src/*. and change the text too. :)

Please update your PR :)

@lgtm-com
Copy link

lgtm-com bot commented Oct 19, 2022

This pull request fixes 2 alerts when merging ba45c4c into 957273f - view on LGTM.com

fixed alerts:

  • 2 for Clear-text logging of sensitive information

@seokho-son
Copy link
Member

ref #1217

@seokho-son seokho-son changed the title Update provisioning.go Change func name CorePostMcisVm into CreateMcisVm Oct 19, 2022
@seokho-son
Copy link
Member

Ping @Rohit-R2000 :)

@seokho-son
Copy link
Member

Hi @Rohit-R2000
Could you check the review ?

@lgtm-com
Copy link

lgtm-com bot commented Oct 26, 2022

This pull request fixes 2 alerts when merging ec2bf47 into 08a2606 - view on LGTM.com

fixed alerts:

  • 2 for Clear-text logging of sensitive information

@seokho-son
Copy link
Member

I updated this PR myself as the maintainer to resolve build failure.
Now, LGTM :)

Thank @Rohit-R2000

@seokho-son seokho-son merged commit b5cf84d into cloud-barista:main Nov 1, 2022
@seokho-son seokho-son added the hacktoberfest-accepted hacktoberfest-accepted label Nov 1, 2022
@seokho-son
Copy link
Member

@all-contributors please add @Rohit-R2000 for code

@allcontributors
Copy link
Contributor

@seokho-son

I've put up a pull request to add @Rohit-R2000! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted hacktoberfest-accepted
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants