[FEAT BRANCH] Add argilla-sdk project
#4891
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
for more information, see https://pre-commit.ci
…lla into feat/add-new-sdk-project
frascuchon
force-pushed
the
feat/add-new-sdk-project
branch
from
b2984fd
to
3768e3b
Compare
frascuchon
force-pushed
the
feat/add-new-sdk-project
branch
from
3768e3b
to
2e31511
Compare
frascuchon
changed the title
frascuchon
changed the title
argilla-sdk project
Closed
|
The URL of the deployed environment for this PR is https://argilla-quickstart-pr-4891-ki24f765kq-no.a.run.app |
frascuchon
commented
Jun 3, 2024
| # Repository | ||
| repo_name: argilla-io/argilla-python | ||
| repo_url: https://github.com/argilla-io/argilla-python | ||
| edit_uri: edit/main/docs/ |
There was a problem hiding this comment.
I saw this line in the latest sync. Should this affect the current project setup? In that case, should we comment until we have merged the project into the default/develop/main one? cc @davidberenstein1957 @burtenshaw
|
I think we should review also some changes in docs: https://github.com/argilla-io/argilla/actions/runs/9352932120/job/25742223903?pr=4891#step:10:39 cc @davidberenstein1957 @sdiazlor |
This PR just applies some minimal changes for the argilla SDK docs generation: - Freeze the default branch to `feat/v2.0.0` - Launch a docs generation for docs branches starting with `docs/**` - Remove PR-related events from triggers.
Closed
sdiazlor
approved these changes
Jun 4, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR setup the argilla-sdk project. This PR only copy the code base from the argilla-python repository.
No changes in package definition or docs have been applied yet.UPDATED: The only change is this line which raise an ruff error when applying commit hooks.Update
The code on this PR won't work with the server version on
feat/v2.0.0until we resolve SDK tasks for the issue #4773. For the sake of the review complexity, I would prefer to tackle this migration as a separate PR.After those changes, the new SDK will still be fully compatible with argilla server 1.29.0.
Closes #4888
Closes #4895
Type of change
(Please delete options that are not relevant. Remember to title the PR according to the type of change)
How Has This Been Tested
(Please describe the tests that you ran to verify your changes. And ideally, reference
tests)Checklist
CHANGELOG.mdfile (See https://keepachangelog.com/)