Remove redundant err check (#2541)

## Description

<!-- Insert PR description-->

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

- [ ] ✅ Yes, it's included
- [ ] 🕐 Yes, but in a later PR
- [x] ⛔ No 

## Type of change

<!--- Please check the type of change your PR introduces: --->
- [ ] 🌻 Feature
- [ ] 🐛 Bugfix
- [ ] 🗺️ Documentation
- [ ] 🤖 Test
- [ ] 💻 CI/Deployment
- [x] 🧹 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] 💪 Manual
- [ ] ⚡ Unit test
- [ ] 💚 E2E

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 {