Skip to content

fix: remove duplicate role lookup branch - #7018

Open
vjymisal0 wants to merge 1 commit into
devtron-labs:mainfrom
vjymisal0:fix/issue-6031
Open

fix: remove duplicate role lookup branch#7018
vjymisal0 wants to merge 1 commit into
devtron-labs:mainfrom
vjymisal0:fix/issue-6031

Conversation

@vjymisal0

Copy link
Copy Markdown

Summary

  • Remove a redundant if/else branch in role lookup where both branches returned the same value.
  • Let unmatched inputs fall through to the existing return path without changing behavior.

Fixes #6031

Validation

  • git diff --check
  • go test ./pkg/auth/user/repository (interrupted after several minutes with no output in this environment)

@bito-code-review

Copy link
Copy Markdown

Bito Review Skipped - Source Branch Not Found

Bito didn't review this change because the pull request is no longer valid. It may have been merged, or the source/target branch may no longer exist.

@sonarqubecloud

sonarqubecloud Bot commented Sep 2, 2026

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Both branches of if-then-else are identical

1 participant