From 727416c6b054cefd313928b7006a9e3c3ca7976f Mon Sep 17 00:00:00 2001
From: Abin Simon <abinsimon10@gmail.com>
Date: Thu, 16 Feb 2023 13:04:58 +0530
Subject: [PATCH] Remove redundant err check (#2541)

## Description

<!-- Insert PR description-->

## Does this PR need a docs update or release note?

- [ ] :white_check_mark: Yes, it's included
- [ ] :clock1: Yes, but in a later PR
- [x] :no_entry: No

## Type of change

<!--- Please check the type of change your PR introduces: --->
- [ ] :sunflower: Feature
- [ ] :bug: Bugfix
- [ ] :world_map: Documentation
- [ ] :robot: Test
- [ ] :computer: CI/Deployment
- [x] :broom: Tech Debt/Cleanup

## Issue(s)

<!-- Can reference multiple issues. Use one of the following "magic words" - "closes, fixes" to auto-close the Github issue. -->
* #<issue>

## Test Plan

<!-- How will this be tested prior to merging.-->
- [x] :muscle: Manual
- [ ] :zap: Unit test
- [ ] :green_heart: E2E
---
 src/pkg/repository/repository.go | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/src/pkg/repository/repository.go b/src/pkg/repository/repository.go
index 2a2ac34ee..272861d31 100644
--- a/src/pkg/repository/repository.go
+++ b/src/pkg/repository/repository.go
@@ -227,12 +227,9 @@ func ConnectAndSendConnectEvent(ctx context.Context,
 	}
 
 	r := repo.(*repository)
+	r.Bus.Event(ctx, events.RepoConnect, nil)
 
-	if err == nil {
-		r.Bus.Event(ctx, events.RepoConnect, nil)
-	}
-
-	return r, err
+	return r, nil
 }
 
 func (r *repository) Close(ctx context.Context) error {