Skip to content

Conversation

bigcat88
Copy link
Contributor

@bigcat88 bigcat88 commented Oct 5, 2025

  1. Removed non needed doc string declaration inside one of the moonvalley function.
  2. Moved Refract nodes away from using non recommended type to isinstance

note: during testing of Recraft nodes found that some of them are currently broken by the transition to async - fix will come in a separate PR with description of problem.

@bigcat88 bigcat88 requested a review from Kosinkadink as a code owner October 5, 2025 06:37
@bigcat88
Copy link
Contributor Author

bigcat88 commented Oct 5, 2025

+label: Core

@bigcat88
Copy link
Contributor Author

bigcat88 commented Oct 5, 2025

PR with fix for ReCraft is ready and tested, will open after merge of this one (they will conflict with each other)

@comfy-pr-bot comfy-pr-bot added the Core Core team dependency label Oct 5, 2025
Copy link
Collaborator

@Kosinkadink Kosinkadink left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

loosk good, merging

@Kosinkadink Kosinkadink merged commit 6ae3515 into comfyanonymous:master Oct 6, 2025
13 checks passed
@bigcat88 bigcat88 deleted the fix/api-nodes/pylint-new-3-rules branch October 7, 2025 05:49
toxicwind pushed a commit to toxicwind/ComfyUI that referenced this pull request Oct 12, 2025
gmaOCR pushed a commit to gmaOCR/ComfyUI that referenced this pull request Oct 14, 2025
adlerfaulkner pushed a commit to LucaLabsInc/ComfyUI that referenced this pull request Oct 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Core Core team dependency

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants