Skip to content

Conversation

@alokdangre
Copy link
Contributor

Add CLI documentation to README, include AskRequest and AskResponse models, and update .gitignore for venv.
I have checked all cli commands its working correctly also i have added readme for you to test it

@alokdangre
Copy link
Contributor Author

@Acuspeedster @juntao can you plzz test it
also let me know if anything else is needed
plzz review it

Copy link
Collaborator

@Acuspeedster Acuspeedster left a comment

Choose a reason for hiding this comment

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

Thanks for adding the CLI functionality 🙌, that’s a great addition!
To make sure these commands are always verified, could you also add a CI pipeline step (e.g., GitHub Actions) that runs and tests the new CLI commands?
That way we’ll automatically catch regressions if future changes break the CLI.

@alokdangre
Copy link
Contributor Author

alokdangre commented Aug 26, 2025

@Acuspeedster i will complete it by tomorrow, still so many fixes are left

Copy link
Collaborator

@Acuspeedster Acuspeedster left a comment

Choose a reason for hiding this comment

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

@juntao It looks good to me.

@juntao juntao merged commit 9e946e1 into cardea-mcp:main Aug 28, 2025
2 checks passed
@juntao
Copy link
Contributor

juntao commented Aug 28, 2025

Thanks!

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.

3 participants