Skip to content

python(feat): annotations resource - #792

Merged
brandonshippy-sift merged 13 commits into
mainfrom
python/annotations-resource
Sep 26, 2026
Merged

brandonshippy-sift merged 13 commits into
mainfrom
python/annotations-resource

Conversation

@brandonshippy-sift

@brandonshippy-sift brandonshippy-sift commented Sep 17, 2026 •

Copy link
Copy Markdown
Contributor

Description

Adds an annotations resource to sift_client, covering data reviews, phases, and the review history.

Annotation and all associated classes and enums are made.
Implements get, list_, find, create, update, assign_to_user, set_open, set_failed, set_accepted, archive/unarchive, add_comment

Validation

  • End to End testing with manual script written to test each new API feature.
  • Verified against a local stack, including the annotation rendering in Explore

@github-actions

github-actions Bot commented Sep 17, 2026 •

Copy link
Copy Markdown
Contributor

Python docs preview: https://sift-stack.github.io/sift/python/pr-792/

Deployed from 06947de. The link may take up to a minute to become live as GitHub Pages propagates.

@brandonshippy-sift
brandonshippy-sift marked this pull request as ready for review September 17, 2026 20:25
Comment thread python/lib/sift_client/resources/annotations.py
Comment thread python/lib/sift_client/resources/annotations.py Outdated
Comment thread python/lib/sift_client/resources/annotations.py Outdated
Comment thread python/lib/sift_client/resources/annotations.py Outdated
Comment thread python/lib/sift_client/sift_types/annotation.py Outdated
Comment thread python/lib/sift_client/_internal/low_level_wrappers/annotations.py Outdated
Comment thread python/lib/sift_client/sift_types/annotation.py
Comment thread python/lib/sift_client/sift_types/annotation.py Outdated
Comment thread python/lib/sift_client/sift_types/annotation.py Outdated
Comment thread python/lib/sift_client/sift_types/annotation.py Outdated
Comment thread python/lib/sift_client/sift_types/annotation.py Outdated
Comment thread python/lib/sift_client/sift_types/annotation.py Outdated
Comment thread python/lib/sift_client/sift_types/annotation.py Outdated
Comment thread python/lib/sift_client/sift_types/annotation.py Outdated
Comment thread python/lib/sift_client/sift_types/annotation.py Outdated
Comment thread python/lib/sift_client/sift_types/annotation.py Outdated
Comment thread python/lib/sift_client/resources/annotations.py Outdated
Comment thread python/lib/sift_client/resources/annotations.py Outdated
Comment thread python/lib/sift_client/resources/annotations.py
Comment thread python/lib/sift_client/resources/annotations.py Outdated
Comment thread python/lib/sift_client/resources/annotations.py Outdated
Comment thread python/lib/sift_client/resources/annotations.py Outdated
Comment thread python/lib/sift_client/resources/annotations.py Outdated
Comment thread python/lib/sift_client/resources/annotations.py
Comment thread python/lib/sift_client/sift_types/annotation.py Outdated

@alexluck-sift alexluck-sift left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Three comments. Also worth filing a ticket for the name: str | Iterable[str] collapse across the other twelve resources, so this PR isn't the only place with the new shape.

Comment thread python/lib/sift_client/resources/annotations.py
Comment thread python/lib/sift_client/resources/annotations.py Outdated
Comment thread python/lib/sift_client/resources/annotations.py
Comment thread python/lib/sift_client/resources/annotations.py Outdated
Comment thread python/lib/sift_client/sift_types/annotation.py Outdated
alexluck-sift
alexluck-sift previously approved these changes Sep 25, 2026
Comment thread python/lib/sift_client/sift_types/annotation.py Outdated

@wei-sift wei-sift left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

@brandonshippy-sift
brandonshippy-sift merged commit 82c0818 into main Sep 26, 2026
26 checks passed
@brandonshippy-sift
brandonshippy-sift deleted the python/annotations-resource branch September 26, 2026 00:09
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