CI version matrix and celery timeout bounds for large file parsing#144
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (5)
✅ Files skipped from review due to trivial changes (4)
📝 WalkthroughWalkthroughThe PR adds configurable soft and hard time limits for add-or-update Celery tasks, maps soft timeouts to a typed endpoint error, and updates CI and package metadata for Python 3.14. ChangesCelery add-or-update task timeouts
Python 3.14 CI support
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related issues
Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
henry0816191
left a comment
There was a problem hiding this comment.
please update related documentations such as docs/boost-endpoint-api.md, readme.md, .github/WORKFLOWS.md and so on
Close #131, close #133.
Summary by CodeRabbit
BOOST_TASK_SOFT_TIME_LIMITandBOOST_TASK_TIME_LIMIT(with defaults and validation).task_timeouterror with relevant metadata.