-
Notifications
You must be signed in to change notification settings - Fork 1
Feat: ENG-25425: Add upload endpoints #59
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
base: master
Are you sure you want to change the base?
Conversation
Add upload endpoints
bnlucas
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The previous release of this was v3.0.2 https://pypi.org/project/lilt-python/. We need to check with @zackigoldberg (#58) as it was released in June.
Update version
Update pipeline
Add missing dependencies
Fix pipeline
|
@mkante-lilt didn't we discuss at standup that we'll release 3.0.3 under the same name? |
Update review
|
@yanwang-lilt @bnlucas I upgrade to |
Exclude workflow tests
Fix pipeline
| pip install git+https://github.com/lilt/lilt-python.git | ||
| pip install git+https://github.com/GIT_USER_ID/GIT_REPO_ID.git | ||
| ``` | ||
| (you may need to run `pip` with root permission: `sudo pip install git+https://github.com/lilt/lilt-python.git`) | ||
| (you may need to run `pip` with root permission: `sudo pip install git+https://github.com/GIT_USER_ID/GIT_REPO_ID.git`) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This needs to remain lilt/lilt-python
| - API version: v3.0.3 | ||
| - Package version: 3.1.0 | ||
| - Generator version: 7.17.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The package version should match the API version.
| git_user_id="lilt" | ||
| git_user_id="GIT_USER_ID" | ||
| echo "[INFO] No command line input provided. Set \$git_user_id to $git_user_id" | ||
| fi | ||
|
|
||
| if [ "$git_repo_id" = "" ]; then | ||
| git_repo_id="lilt-python" | ||
| git_repo_id="GIT_REPO_ID" | ||
| echo "[INFO] No command line input provided. Set \$git_repo_id to $git_repo_id" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should stay as lilt-python
Changes
This pull request updates the
README.mdand documentation files for the Python client of the LILT API. The main changes include updating the package and API version information, adding documentation for new API endpoints and models (especially around Segments and Uploads), and removing several outdated or redundant model documentation files.Key changes:
Documentation and API Reference Updates
README.mdfromlilt-pythontolilt-client-python, revised the API support and authentication instructions, and updated installation instructions to use placeholder GitHub repository values. [1] [2]v3.0.1, package version to0.7.0, and generator version to7.17.0in theREADME.md.SegmentsApiandUploadsApito the API reference table inREADME.md, reflecting expanded functionality for segment and file upload management.README.md. [1] [2] [3]New Model Documentation
CancelMultipartUpload200Response,CompleteMultipartUpload200Response, and others. [1] [2]Cleanup of Outdated Model Documentation
docs/directory, includingAnnotation.md,ApiRoot.md,AssignmentDetails.md,AssignmentError.md,AutoAssignmentParameters.md,AutoAssignmentResponse.md,AutoAssignmentResponseAssignments.md,AutoAssignmentResponseErrors.md,BadRequest.md,Comment.md,CommentBody.md, andCommentDeleteResponse.md. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12]These updates ensure the documentation is current with the latest API capabilities and remove obsolete references, making the client library easier to use and maintain.