Skip to content

Commit

Permalink
Merge pull request #170 from drone/remove-double-invocation
Browse files Browse the repository at this point in the history
remove double invocation of convertRepository
  • Loading branch information
TP Honey authored Apr 22, 2022
2 parents ed4abc6 + 50a50a8 commit 696d049
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions scm/driver/github/repo.go
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ func (s *RepositoryService) Find(ctx context.Context, repo string) (*scm.Reposit
if convertedRepo == nil {
return nil, res, errors.New("GitHub returned an unexpected null repository")
}
return convertRepository(out), res, err
return convertedRepo, res, err
}

// FindHook returns a repository hook.
Expand All @@ -94,7 +94,7 @@ func (s *RepositoryService) FindPerms(ctx context.Context, repo string) (*scm.Pe
if convertedRepo == nil {
return nil, res, errors.New("GitHub returned an unexpected null repository")
}
return convertRepository(out).Perm, res, err
return convertedRepo.Perm, res, err
}

// List returns the user repository list.
Expand Down

0 comments on commit 696d049

Please sign in to comment.