Skip to content

Commit 0a76d0b

Browse files
committed
PR cleanups: Fix govet errors and update the README
1 parent 8934b9e commit 0a76d0b

2 files changed

Lines changed: 6 additions & 3 deletions

File tree

README.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,10 @@ The format written is the one defined by the
77
[git-appraise code review system](https://github.com/google/git-appraise), so pull
88
requests that are mirrored using this tool can be reviewed using git-appraise.
99

10+
## Disclaimer
11+
12+
This is not an officially supported Google product.
13+
1014
## Organization
1115

1216
There are 3 packages in this repo:

app/admin/operations.go

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -190,9 +190,8 @@ func validate(ctx context.Context, user, repo string) {
190190

191191
log.Infof(ctx, "Validated repo %s/%s", user, repo)
192192

193-
var remoteRepo *github.Repository
194193
err = retry(ctx, func() (resp *github.Response, err error) {
195-
remoteRepo, resp, err = githubClient.Repositories.Get(ctx, user, repo)
194+
_, resp, err = githubClient.Repositories.Get(ctx, user, repo)
196195
return
197196
})
198197

@@ -307,7 +306,7 @@ func deactivate(ctx context.Context, userName, repoName string) {
307306
errorf("Can't delete webhook: %s", err.Error())
308307
// Keep going; we should still delete the repository data
309308
} else {
310-
log.Infof(ctx, "Deleting hook for repository %s/%s succeeded")
309+
log.Infof(ctx, "Deleting hook for repository %s/%s succeeded", userName, repoName)
311310
}
312311

313312
log.Infof(ctx, "Deleting repository data for %s/%s", userName, repoName)

0 commit comments

Comments
 (0)