Home Original page

fix: Correct body in `EnterpriseService.InitialConfig` by alexandear · Pull Request #3763 · google/go-github

This PR fixes the body in the EnterpriseService.InitialConfig endpoint. From the documentation:

image

Before the PR, the request body is:

{"License":"1234-1234","Password":"password"}

After:

{"license":"1234-1234","password":"password"}

The bug was found with the help of musttag linter:

$ ./script/lint.sh
linting .
github/enterprise_manage_ghes_config_test.go:607:52: the given struct should be annotated with the `json` tag (musttag)
                assertNilError(t, json.NewDecoder(r.Body).Decode(v))
                                                                 ^
github/timestamp_test.go:132:28: the given struct should be annotated with the `json` tag (musttag)
                out, err := json.Marshal(tc.data)
                                         ^
github/timestamp_test.go:199:30: the given struct should be annotated with the `json` tag (musttag)
                bytes, err := json.Marshal(tc.data)
                                           ^
3 issues:
* musttag: 3

So, the PR enables it as well.