| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
There was a problem hiding this comment.
LGTM. Clean, correct implementation.
The nil-guard on overrides is fine — passing nil vs &clientcmd.ConfigOverrides{CurrentContext: ""} are both equivalent (no context override), but the explicit nil check makes intent clearer.
Passing &clientcmd.ConfigOverrides{CurrentContext: options.Context} unconditionally would also work and simplify the code slightly, but this is a nit — the current approach is readable.
Flag name --context matches kubectl convention. ✅
Change is symmetric between backend and konnector. ✅
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Summary
What Type of PR Is This?
/kind feature
Related Issue(s)
Fixes 479
Release Notes