Skip to content

Conversation

@mmacpherson
Copy link
Owner

Summary

  • Adds TimeoutError to the exception handling in get_latest_lucide_version()
  • Prevents the update-lucide workflow from failing when the GitHub API times out

Test plan

  • Pre-commit hooks pass
  • Tests pass
  • Re-run the update-lucide workflow after merging

🤖 Generated with Claude Code

The GitHub API call can timeout, raising TimeoutError which is a
subclass of OSError, not URLError. This was causing the update-lucide
workflow to fail instead of gracefully handling the network issue.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@mmacpherson mmacpherson merged commit 82440d0 into main Dec 22, 2025
6 checks passed
@mmacpherson mmacpherson deleted the fix/handle-timeout-error branch December 22, 2025 19:54
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