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 receiver name in client and comments #379

Merged
merged 15 commits into from
Oct 6, 2021

Conversation

crashiura
Copy link
Contributor

This was a trivial fix so there is no related Issue
I added comments and fixed receiver name in client.

@vmwclabot
Copy link
Member

@crashiura, you must sign our contributor license agreement before your changes are merged. Click here to sign the agreement. If you are a VMware employee, read this for further instruction.

@crashiura crashiura changed the title Fix receiver name in client and pretty code Fix receiver name in client and comments May 28, 2021
Copy link
Collaborator

@Didainius Didainius left a comment

Choose a reason for hiding this comment

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

Thanks for this PR.

Just a question - did you run any test on it. I saw there were a few shuffles on error handling and wondering if it has any consequences.

crashiura and others added 9 commits May 29, 2021 00:48
@crashiura
Copy link
Contributor Author

Thank you for comments, I ran unit tests

Copy link
Collaborator

@Didainius Didainius left a comment

Choose a reason for hiding this comment

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

Looks like it does not impacts tests after run.
Thanks.

Also - please do add a comment to CHANGELOG.md before other team members review it.

@crashiura
Copy link
Contributor Author

I added comment in changelog.

@vmwclabot
Copy link
Member

@crashiura, VMware has approved your signed contributor license agreement.

Co-authored-by: lvirbalas <[email protected]>
Copy link
Collaborator

@lvirbalas lvirbalas 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 for contributing.

@crashiura
Copy link
Contributor Author

@dataclouder check pls

crashiura and others added 2 commits June 17, 2021 12:07
Co-authored-by: Giuseppe Maxia <[email protected]>
Co-authored-by: Giuseppe Maxia <[email protected]>
@crashiura
Copy link
Contributor Author

@dataclouder done

@crashiura
Copy link
Contributor Author

@dataclouder Hi, check pls

@dataclouder dataclouder merged commit 79873f4 into vmware:master Oct 6, 2021
dataclouder pushed a commit to dataclouder/go-vcloud-director that referenced this pull request Oct 6, 2021
dataclouder added a commit that referenced this pull request Oct 8, 2021
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.

5 participants