hashicorp / boundary

Boundary enables identity-based access management for dynamic infrastructure.
https://boundaryproject.io
Other
3.86k stars 289 forks source link

Both branches of `if-then-else` are identical #5205

Open chavacava opened 1 month ago

chavacava commented 1 month ago

https://github.com/hashicorp/boundary/blob/1f0bad4d482c66a75da7b407f56fd3437aec85a7/internal/iam/repository_principal_role_test.go#L144-L148

https://github.com/hashicorp/boundary/blob/1f0bad4d482c66a75da7b407f56fd3437aec85a7/internal/iam/repository_principal_role_test.go#L153-L157

Found with revive

bgajjala8 commented 4 weeks ago

Hi! I’ve submitted a fix to clean up the redundant code

https://github.com/hashicorp/boundary/pull/5211