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

govc: camelCase json output #3247

Merged
merged 1 commit into from
Sep 28, 2023
Merged

govc: camelCase json output #3247

merged 1 commit into from
Sep 28, 2023

Conversation

dougm
Copy link
Member

@dougm dougm commented Sep 27, 2023

PR #3007 added camelCase json tags for generated types BREAKING: This change adds camelCase (lower-case 1st character) json tags for govc types

Fixes #3243
Fixes #3242
Fixes #3241

Tags were added using:

% gomodifytags -w -transform camelcase -add-tags json -all -skip-unexported -file ...

PR vmware#3007 added camelCase json tags for generated types
BREAKING: This change adds camelCase (lower-case 1st character) json tags for govc types

Fixes vmware#3243
Fixes vmware#3242
Fixes vmware#3241
Copy link
Member

@akutz akutz left a comment

Choose a reason for hiding this comment

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

Lgtm!

@dougm dougm merged commit 0197ef5 into vmware:main Sep 28, 2023
10 checks passed
@dougm dougm deleted the govc-json-camelCase branch September 28, 2023 17:13
@jsvoboda-prgs jsvoboda-prgs mentioned this pull request Nov 13, 2023
dougm added a commit to dougm/govmomi that referenced this pull request Feb 8, 2024
BREAKING: follow up to json tags missed in PR vmware#3007 and PR vmware#3247

Closes vmware#3318
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants