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

Add serve command #2040

Merged
merged 4 commits into from
Nov 8, 2024
Merged

Add serve command #2040

merged 4 commits into from
Nov 8, 2024

Conversation

meatballhat
Copy link
Contributor

@meatballhat meatballhat commented Oct 31, 2024

to run an HTTP server command very similar to what is used in Replicate production. By making this command more within reach, my hope is that not only is it easier and more clear to run Cog in its HTTP server mode, but also that more Cog users will build familiarity with and opinions about the HTTP server interface.

Connected to PLAT-259

to run the HTTP server command very similar to what is used in Replicate
production. By making this command more within reach, my hope is that
more Cog users will build familiarity with and opinions about the HTTP
server interface.

Connected to PLAT-259
@meatballhat meatballhat requested a review from a team October 31, 2024 17:01
Copy link
Member

@erbridge erbridge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks like a good addition to me.

pkg/cli/root.go Outdated Show resolved Hide resolved
Copy link
Member

@nickstenning nickstenning left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like it!

Let's run this past @bfirsh and/or @superhighfives as a quick reasonableness check? Otherwise LGTM.

@superhighfives
Copy link

Agreed that this makes sense—I think my only request would be to add the details to the README.

@bfirsh
Copy link
Member

bfirsh commented Nov 8, 2024

Love it!

@meatballhat meatballhat merged commit e46c4f3 into main Nov 8, 2024
19 checks passed
@meatballhat meatballhat deleted the serve-command branch November 8, 2024 19:10
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.

5 participants