-
Notifications
You must be signed in to change notification settings - Fork 446
[fix] Weekly release workflow adjustments #7060
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
- Rename file: weekly-comfyui-release.yaml → release-weekly-comfyui.yaml - Update name: "Weekly ComfyUI Release" → "Release: Weekly ComfyUI" - Follows pattern of other release-* workflows
- Fetch latest upstream/master before making changes - Ensures PR only shows requirements.txt diff, not stale commits - Does not modify fork's master branch (only pushes automation branch)
|
Warning Rate limit exceeded@christian-byrne has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 4 minutes and 13 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe weekly ComfyUI release workflow is renamed for consistency and a new synchronization step is added to fetch and checkout the upstream repository's master branch before performing subsequent release operations. Changes
Possibly related PRs
✨ Finishing touches🧪 Generate unit tests (beta)
Comment |
🎨 Storybook Build Status✅ Build completed successfully! ⏰ Completed at: 11/30/2025, 10:12:13 PM UTC 🔗 Links🎉 Your Storybook is ready for review! |
🎭 Playwright Test Results⏰ Completed at: 11/30/2025, 10:21:47 PM UTC 📈 Summary
📊 Test Reports by Browser
🎉 Click on the links above to view detailed test results for each browser configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
.github/workflows/release-weekly-comfyui.yaml(2 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)
- GitHub Check: setup
- GitHub Check: test
- GitHub Check: collect
- GitHub Check: lint-and-format
🔇 Additional comments (1)
.github/workflows/release-weekly-comfyui.yaml (1)
2-2: Workflow name aligns with convention.The rename from "Weekly ComfyUI Release" to "Release: Weekly ComfyUI" matches the pattern of other
release-*workflows as intended.
Bundle Size ReportSummary
Category Glance Per-category breakdownApp Entry Points — 3.18 MB (baseline 3.18 MB) • ⚪ 0 BMain entry bundles and manifests
Graph Workspace — 950 kB (baseline 950 kB) • ⚪ 0 BGraph editor runtime, canvas, workflow orchestration
Views & Navigation — 6.54 kB (baseline 6.54 kB) • ⚪ 0 BTop-level views, pages, and routed surfaces
Panels & Settings — 298 kB (baseline 298 kB) • ⚪ 0 BConfiguration panels, inspectors, and settings screens
UI Components — 139 kB (baseline 139 kB) • ⚪ 0 BReusable component library chunks
Data & Services — 12.5 kB (baseline 12.5 kB) • ⚪ 0 BStores, services, APIs, and repositories
Utilities & Hooks — 2.94 kB (baseline 2.94 kB) • ⚪ 0 BHelpers, composables, and utility bundles
Vendor & Third-Party — 8.56 MB (baseline 8.56 MB) • ⚪ 0 BExternal libraries and shared vendor chunks
Other — 3.84 MB (baseline 3.84 MB) • ⚪ 0 BBundles that do not match a named category
|
- Enable strict mode with set -euo pipefail - Check exit status of git fetch and checkout - Log clear error messages on failure - Prevents subsequent steps from running on stale checkout
Follow-up adjustments to the weekly ComfyUI release automation workflow.
Changes
Rename workflow to follow conventions
weekly-comfyui-release.yaml→release-weekly-comfyui.yamlrelease-*workflowsSync fork with upstream before creating PR
Testing
After merge, can test via manual workflow dispatch in Actions tab.
┆Issue is synchronized with this Notion page by Unito