feat: cli initial setup

deepset-ai / deepset-cloud-sdk

A Python SDK to interact with deepset Cloud

Apache License 2.0

9 stars 2 forks source link

feat: cli initial setup #21

Closed ArzelaAscoIi closed 1 year ago

ArzelaAscoIi commented 1 year ago

Related Issues

add cli commands for uploading files

changes

add cli for uploading paths and files

Tests

unit tests (no need to have integration tests since this is already integration tested)

Notes

I did NOT add check if the package.toml works as expected and users can run deepset_cloud_cli <command>

Checklist

[ ] I have updated the referenced issue with new insights and changes
[ ] If this is a code change, I have added unit tests
[ ] I've used the conventional commit specification for my PR title
[ ] I updated the docstrings
[ ] If this is a code change, I added meaningful logs and prepared Datadog visualizations and alerts

github-actions[bot] commented 1 year ago

Coverage report

The coverage rate went from 85% to 86% :arrow_up: The branch rate is 76%.

97% of new lines are covered.

Diff Coverage details (click to unfold)

### deepset_cloud_sdk/cli.py `100%` of new lines are covered (`100%` of the complete file). ### deepset_cloud_sdk/api/config.py `93%` of new lines are covered (`94%` of the complete file). Missing lines: `37`

ArzelaAscoIi commented 1 year ago

Looks really good. The CI isn't completely working though. I want to deploy this to the test PyPi so that we can test if it works as intended. I am not sure that everything is set up correctly for us to use it as described in the README

yes. i think there is some path variable missing