Skip to content

Commit

Permalink
Merge pull request #4517 from communitybridge/unicron-fix-lint
Browse files Browse the repository at this point in the history
Fix the linter
  • Loading branch information
mlehotskylf authored Jan 15, 2025
2 parents 28ad779 + 22b3f3e commit e0e9aed
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions cla-backend-go/gerrits/service.go
Original file line number Diff line number Diff line change
Expand Up @@ -136,19 +136,19 @@ func (s service) AddGerrit(ctx context.Context, claGroupID string, projectSFID s

var groupNameCcla, groupNameIcla string
if params.GroupIDIcla != "" {
group, err := s.lfGroup.GetGroup(ctx, params.GroupIDIcla)
if err != nil {
group, err2 := s.lfGroup.GetGroup(ctx, params.GroupIDIcla)
if err2 != nil {
message := fmt.Sprintf("unable to get LDAP ICLA Group: %s", params.GroupIDIcla)
log.WithFields(f).WithError(err).Warnf(message)
log.WithFields(f).WithError(err2).Warnf(message)
return nil, errors.New(message)
}
groupNameIcla = group.Title
}
if params.GroupIDCcla != "" {
group, err := s.lfGroup.GetGroup(ctx, params.GroupIDCcla)
if err != nil {
group, err2 := s.lfGroup.GetGroup(ctx, params.GroupIDCcla)
if err2 != nil {
message := fmt.Sprintf("unable to get LDAP CCLA Group: %s", params.GroupIDCcla)
log.WithFields(f).WithError(err).Warnf(message)
log.WithFields(f).WithError(err2).Warnf(message)
return nil, errors.New(message)
}
groupNameCcla = group.Title
Expand Down

0 comments on commit e0e9aed

Please sign in to comment.