Skip to content

fix(ci): apply lobbying-fetch timeout fix to update-weekly.yml too - #112

Merged
nesanders merged 1 commit into
mainfrom
fix/weekly-workflow-timeout-parity
Aug 22, 2026
Merged

fix(ci): apply lobbying-fetch timeout fix to update-weekly.yml too#112
nesanders merged 1 commit into
mainfrom
fix/weekly-workflow-timeout-parity

Conversation

@nesanders

Copy link
Copy Markdown
Owner

Problem

#111 fixed the 45-min timeout on the lobbying-fetch step, but only in update-data.yml ("Update Data Only"). It missed that update-weekly.yml ("Weekly Update (Data + Charts)") — the workflow that actually failed 3 weeks running (#108, #109, #110) and runs on the Monday schedule — maintains its own duplicate copy of the same fetch steps. That file still had timeout-minutes: 45 and no PYTHONUNBUFFERED.

Running the weekly workflow as-is would just time out again.

Fix

Same change as #111, applied to update-weekly.yml: timeout-minutes: 45 → 90, PYTHONUNBUFFERED: "1". Restores parity between the two files' fetch sections (confirmed identical via diff).

🤖 Generated with Claude Code

#111 fixed the 45-min timeout on the lobbying-fetch step in update-data.yml
("Update Data Only"), but missed that update-weekly.yml ("Weekly Update (Data +
Charts)") — the workflow that actually failed 3 weeks running (#108/#109/#110)
and runs on the Monday schedule — maintains its own duplicate copy of the same
steps. It still had timeout-minutes: 45 and no PYTHONUNBUFFERED.

Same fix, same rationale as #111: timeout-minutes 45 -> 90, PYTHONUNBUFFERED=1.
Restores parity between the two files' fetch sections.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@nesanders
nesanders force-pushed the fix/weekly-workflow-timeout-parity branch from 97e901e to 926cfc8 Compare August 22, 2026 19:03
@nesanders
nesanders merged commit e3441f5 into main Aug 22, 2026
5 checks passed
@nesanders
nesanders deleted the fix/weekly-workflow-timeout-parity branch August 22, 2026 19:04
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.

1 participant