Skip to content

Start Command #828

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

Merged
merged 21 commits into from
Jun 25, 2024
Merged

Start Command #828

merged 21 commits into from
Jun 25, 2024

Conversation

CalebCourier
Copy link
Collaborator

@CalebCourier CalebCourier commented Jun 14, 2024

This PR adds a basic start command to the guardrails cli. This command checks if the guardrails-api module is installed, if it is not, then it installs it (currently a local install until we publish), then it calls the guardrails-api cli's start command. This starts the guardrails-api on localhost:8000.

We also add a new dependency group so users can install with the api from the start if they wish.

Dependencies

#804
guardrails-ai/guardrails-api#44

Cannot be merged until the above two PRs are merged and released.

TODO's

  • Release 0.5.0a2
  • Release guardrails-api
  • Update install and import of guardrails-api
  • Release 0.5.0a3

@CalebCourier CalebCourier requested review from zsimjee, nichwch and a team June 14, 2024 16:38
Base automatically changed from core-schema-impl to 0.5.0-dev June 17, 2024 13:05
Copy link

Start command

2 similar comments
Copy link

Start command

Copy link

Start command

@CalebCourier CalebCourier changed the base branch from 0.5.0-dev to feat/local-server June 18, 2024 16:09
@CalebCourier CalebCourier marked this pull request as ready for review June 24, 2024 23:20
@CalebCourier CalebCourier changed the base branch from feat/local-server to 0.5.0-dev June 24, 2024 23:21
@CalebCourier CalebCourier merged commit 46b27ea into 0.5.0-dev Jun 25, 2024
13 checks passed
@CalebCourier CalebCourier deleted the start-cmd branch June 25, 2024 00:31
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