Skip to content
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

Simplify container restart command #74

Merged
merged 1 commit into from
Feb 7, 2025

Conversation

evankanderson
Copy link
Member

I was annoyed that it was 4 commands to upgrade codegate; this reduces it to 3 commands. It might be possible to make an upgrade button if we can plumb through the docker socket into the container...

Copy link

vercel bot commented Feb 7, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
codegate-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 7, 2025 5:08pm

@danbarr
Copy link
Collaborator

danbarr commented Feb 7, 2025

Good point, I don't know why I didn't do this in the first place. I've just been running both with &&.

@evankanderson If you're up for it, docker stop codegate && docker rm codegate appears in a bunch of spots in the docs and could be replaced too. Or I can do that in my next PR.

@danbarr danbarr self-requested a review February 7, 2025 17:05
@evankanderson
Copy link
Member Author

I can do that.

@evankanderson evankanderson merged commit 1a93f17 into stacklok:main Feb 7, 2025
6 checks passed
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.

2 participants