Skip to content

[86b8jt0u9] Fix pydantic model being passed directly to requests#156

Merged
mpanik merged 1 commit intomainfrom
fix-runs-list-cmd-CU-86b8jt0u9
Feb 27, 2026
Merged

[86b8jt0u9] Fix pydantic model being passed directly to requests#156
mpanik merged 1 commit intomainfrom
fix-runs-list-cmd-CU-86b8jt0u9

Conversation

@mpanik
Copy link
Contributor

@mpanik mpanik commented Feb 27, 2026

❯ uv run omics workbench runs list --show-hidden --page-size 5 --page 0 --max-results 5 | jq '.[].run_id'
"c39c6d87-e4d8-4788-bfa7-7d20b3ca1822"
"9e1f4ff4-b980-451a-826b-6289fc47b159"
"29a450c7-8487-4a61-9e94-2e07098c7c5a"
"05d7eb20-c304-4220-b55d-0116b28e6c21"
"ee8ec3b3-19cd-49ac-970d-3404ef916dc5"

❯ uv run omics workbench runs list --show-hidden --page-size 5 --page 0 --max-results 1 | jq '.[].run_id'
"c39c6d87-e4d8-4788-bfa7-7d20b3ca1822"

❯ uv run omics workbench runs list --show-hidden --page-size 5 --page 0 --max-results 2 | jq '.[].run_id'
"c39c6d87-e4d8-4788-bfa7-7d20b3ca1822"
"9e1f4ff4-b980-451a-826b-6289fc47b159"

❯ uv run omics workbench runs list --show-hidden --page-size 5 --page 1 --max-results 5 | jq '.[].run_id'
"74273bad-c73d-43ea-b011-8b2ae4b80a5e"
"4e527b83-bf9f-4101-bb51-9b9225c4fb2a"
"6a2e24f7-6426-47b4-9fde-93324890e3fd"
"9bb689cf-b4b4-46a2-bc8d-2ee81e84deb0"
"f7ff149e-8023-413f-9734-a1abde8d3145"

❯ uv run omics workbench runs list --show-hidden --page-size 5 --page 1 --max-results 2 | jq '.[].run_id'
"74273bad-c73d-43ea-b011-8b2ae4b80a5e"
"4e527b83-bf9f-4101-bb51-9b9225c4fb2a"

@platform-automation-dnastack

@github-actions
Copy link

github-actions bot commented Feb 27, 2026

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
10742 6036 56% 30% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
dnastack/cli/commands/utils.py 100% 🟢
dnastack/client/collections/client.py 26% 🟢
dnastack/client/workbench/base_client.py 62% 🟢
TOTAL 63% 🟢

updated for commit: 8be8304 by action🐍

Copy link
Contributor

@patmagee patmagee left a comment

Choose a reason for hiding this comment

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

LGTM

@mpanik mpanik merged commit caeba4a into main Feb 27, 2026
6 checks passed
@mpanik mpanik deleted the fix-runs-list-cmd-CU-86b8jt0u9 branch February 27, 2026 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants