Skip to content

docs: remove unused LanguageFilter imports from starting page - #812

Merged
LinoGiger merged 1 commit into
mainfrom
docs/remove-unused-language-filter-imports
Aug 13, 2026
Merged

docs: remove unused LanguageFilter imports from starting page#812
LinoGiger merged 1 commit into
mainfrom
docs/remove-unused-language-filter-imports

Conversation

@RapidPoseidon

Copy link
Copy Markdown
Contributor

The Audio and Text tabs of the docs landing page import LanguageFilter but never use it — leftovers from when these examples used the deprecated order API, which took a filters= argument.

create_compare_job_definition has no filters parameter; in the Jobs model, annotator filtering lives on the audience (audience.filter([LanguageFilter(["en"])]), documented in docs/audiences.md). The dangling import points readers at a parameter that does not exist on the job path.

This came out of investigating why a customer on the latest SDK (3.19.2) reached for create_compare_order instead of a job definition: they needed an English-only audience, and filters= only exists on the deprecated order API. The unused import here is one of the trails leading there.

Docs-only, two lines. Verified with uv run --group docs mkdocs build — builds clean, remaining warnings are pre-existing and unrelated.

🔗 Session: session-af1a7b32

Co-Authored-By: lino <68745352+LinoGiger@users.noreply.github.com>
@LinoGiger
LinoGiger marked this pull request as ready for review August 13, 2026 08:05
@LinoGiger
LinoGiger self-requested a review as a code owner August 13, 2026 08:05
@LinoGiger
LinoGiger merged commit 13a3e32 into main Aug 13, 2026
1 check passed
@LinoGiger
LinoGiger deleted the docs/remove-unused-language-filter-imports branch August 13, 2026 08:05
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.

2 participants