[PR #816] [MERGED] [Cleanup] Minor refactors and comments #1126

Closed
opened 2026-10-31 22:19:43 +00:00 by bsakaguchi · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/Comfy-Org/desktop/pull/816
Author: @webfiltered
Created: 2/4/2025
Status: ✅ Merged
Merged: 2/4/2025
Merged by: @webfiltered

Base: main ← Head: nits-iv


📝 Commits (4)

  • d7edfa5 Remove unused code
  • 836c330 nit - Docs
  • 643a321 [Refactor] Declarative code split
  • d600e92 [Refactor] Move interface to separate file

📊 Changes

4 files changed (+19 additions, -28 deletions)

View changed files

📝 src/desktopApp.ts (+1 -13)
➕ src/infrastructure/interfaces.ts (+12 -0)
📝 src/install/installationManager.ts (+4 -14)
📝 src/main-process/comfyInstallation.ts (+2 -1)

📄 Description

┆Issue is synchronized with this Notion page by Unito


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/Comfy-Org/desktop/pull/816 **Author:** [@webfiltered](https://github.com/webfiltered) **Created:** 2/4/2025 **Status:** ✅ Merged **Merged:** 2/4/2025 **Merged by:** [@webfiltered](https://github.com/webfiltered) **Base:** `main` ← **Head:** `nits-iv` --- ### 📝 Commits (4) - [`d7edfa5`](https://github.com/Comfy-Org/desktop/commit/d7edfa5b72a04bb582f4b8cf00ff7ff090f5b767) Remove unused code - [`836c330`](https://github.com/Comfy-Org/desktop/commit/836c330a20c55e7762e04b37b39014164304553f) nit - Docs - [`643a321`](https://github.com/Comfy-Org/desktop/commit/643a321d0c9d6f5b7a45c7c1cc59cd083e9f4e91) [Refactor] Declarative code split - [`d600e92`](https://github.com/Comfy-Org/desktop/commit/d600e92559820b190c77be5cb56d2c925bb72c17) [Refactor] Move interface to separate file ### 📊 Changes **4 files changed** (+19 additions, -28 deletions) <details> <summary>View changed files</summary> 📝 `src/desktopApp.ts` (+1 -13) ➕ `src/infrastructure/interfaces.ts` (+12 -0) 📝 `src/install/installationManager.ts` (+4 -14) 📝 `src/main-process/comfyInstallation.ts` (+2 -1) </details> ### 📄 Description ┆Issue is synchronized with this [Notion page](https://www.notion.so/PR-816-Cleanup-Minor-refactors-and-comments-1906d73d3650818ab99fde4d99ba15d7) by [Unito](https://www.unito.io) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
bsakaguchi 2026-10-31 22:19:43 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Comfy-Org/desktop#1126
No description provided.