Skip to content

Commit 89b8684

Browse files
author
alex-omophub
committed
Update CI workflow to support multiple branches and manual triggering
1 parent 1c87524 commit 89b8684

2 files changed

Lines changed: 10 additions & 2 deletions

File tree

.github/workflows/ci.yml

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,10 @@ name: CI
22

33
on:
44
push:
5-
branches: [main]
5+
branches: [main, master, develop]
66
pull_request:
7-
branches: [main]
7+
branches: [main, master, develop]
8+
workflow_dispatch: # Allow manual triggering
89

910
jobs:
1011
lint:
@@ -52,6 +53,8 @@ jobs:
5253
token: ${{ secrets.CODECOV_TOKEN }}
5354

5455
integration:
56+
# Only run on push to main/develop, not on PRs (saves ~7-8 min per PR)
57+
if: github.event_name == 'push' || github.event_name == 'workflow_dispatch'
5558
runs-on: ubuntu-latest
5659
steps:
5760
- uses: actions/checkout@v4

tests/conftest.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -172,6 +172,11 @@ def extract_data(result: dict[str, Any] | list[Any], key: str) -> list[Any]:
172172
value = result.get(key)
173173
if isinstance(value, list):
174174
return value
175+
# Handle nested dicts: {"results": {"results": [...]}}
176+
if isinstance(value, dict) and key in value:
177+
nested = value.get(key)
178+
if isinstance(nested, list):
179+
return nested
175180
# Fallback: check 'results' key for batch endpoint backward compatibility
176181
# (production API returns 'results', new API will return 'concepts')
177182
if key == "concepts":

0 commit comments

Comments
 (0)