New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Replace hardcoded URL with Endpoints from context #2199
Conversation
❌ Deploy preview for meshery-docs failed. 🔨 Explore the source changes: b51666b 🔍 Inspect the deploy logs: https://app.netlify.com/sites/meshery-docs/deploys/5ff61f083f4888000826e56b |
While you're updating the cfg package, @anirudhjain75, perhaps, change it's name to "config"... |
Sure @leecalcote |
Hey @leecalcote , Version command is now context aware |
e1b58ae
to
99c3bf4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
A couple conflicts to knock out... |
…. Rework cfg.go to config.go with required design and architecture changes to support context. Signed-off-by: Anirudh Jain <anirudh.jain@outlook.com>
Signed-off-by: Anirudh Jain <anirudh.jain@outlook.com>
Signed-off-by: Anirudh Jain <anirudh.jain@outlook.com>
Signed-off-by: Anirudh Jain <anirudh.jain@outlook.com>
2e008b5
to
b51666b
Compare
Signed-off-by: Anirudh Jain anirudh.jain@outlook.com