docs(readme): add deprecation banner pointing to Comfy-Org/Comfy-Desktop#1730
docs(readme): add deprecation banner pointing to Comfy-Org/Comfy-Desktop#1730deepme987 wants to merge 2 commits into
Conversation
This repo has been superseded by the rewrite at Comfy-Org/Comfy-Desktop. Existing 0.9.x users will get an in-app update to the new version; new downloads route through dl.comfy.org. The rest of the README is kept as historical reference.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughWalkthroughThe README is updated with a prominent ChangesDocumentation Update
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
…rips browser safe-download)
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@README.md`:
- Line 7: Update the README sentence that currently guarantees automatic in-app
updates for "Existing 0.9.x users" so it reflects actual updater behavior:
change the wording to make the in-app update conditional on the user's
auto-update setting (Comfy-Desktop.AutoUpdate) and indicate that users with
auto-updates enabled will receive the update while others must update manually
or enable auto-updates; edit the line containing "Existing 0.9.x users"
accordingly.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
| > **This repository is no longer the home of Comfy Desktop.** The app has been rewritten and now lives at **[Comfy-Org/Comfy-Desktop](https://github.com/Comfy-Org/Comfy-Desktop)**. | ||
| > | ||
| > - **New downloads:** [dl.comfy.org](https://dl.comfy.org) | ||
| > - **Existing 0.9.x users:** your app will receive an in-app update to the new version as we roll out — no action needed. |
There was a problem hiding this comment.
Soften the in-app update guarantee to match actual updater behavior.
Line 7 currently guarantees automatic migration for all 0.9.x users, but updater behavior depends on user settings (Comfy-Desktop.AutoUpdate). Please make this conditional.
Suggested wording
-> - **Existing 0.9.x users:** your app will receive an in-app update to the new version as we roll out — no action needed.
+> - **Existing 0.9.x users:** if auto-update is enabled, your app should receive an in-app update to the new version as rollout progresses.📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| > - **Existing 0.9.x users:** your app will receive an in-app update to the new version as we roll out — no action needed. | |
| > - **Existing 0.9.x users:** if auto-update is enabled, your app should receive an in-app update to the new version as rollout progresses. |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@README.md` at line 7, Update the README sentence that currently guarantees
automatic in-app updates for "Existing 0.9.x users" so it reflects actual
updater behavior: change the wording to make the in-app update conditional on
the user's auto-update setting (Comfy-Desktop.AutoUpdate) and indicate that
users with auto-updates enabled will receive the update while others must update
manually or enable auto-updates; edit the line containing "Existing 0.9.x users"
accordingly.
Adds a GitHub-alert banner at the very top of the README pointing visitors to the rewrite at Comfy-Org/Comfy-Desktop.
The rest of the README is kept as historical reference — no content removed.
9-line addition.
Summary by CodeRabbit