Skip to content

bug fix: remove print and logger statements#217

Merged
srijanpatel merged 2 commits intomainfrom
feat/workflow-as-code
Mar 14, 2025
Merged

bug fix: remove print and logger statements#217
srijanpatel merged 2 commits intomainfrom
feat/workflow-as-code

Conversation

@srijanpatel
Copy link
Collaborator

@srijanpatel srijanpatel commented Mar 14, 2025

This pull request includes a small change to the backend/pyspur/nodes/decorator.py file. The change removes debug logging and a print statement from the run method to clean up the code.


Important

Remove debug logging and print statement from run method in decorator.py for code cleanup.

  • Code Cleanup:
    • Removed debug logging and print statement from run method in decorator.py to clean up the code.

This description was created by Ellipsis for 444588f. It will automatically update as commits are pushed.

@srijanpatel srijanpatel merged commit 8157884 into main Mar 14, 2025
1 check passed
Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

👍 Looks good to me! Reviewed everything up to 444588f in 32 seconds

More details
  • Looked at 13 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 2 drafted comments based on config settings.
1. backend/pyspur/nodes/decorator.py:230
  • Draft comment:
    Removed debug logging and print statements. Ensure that if you need debugging in the future, consider a toggleable debug flag or proper log-level configuration.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50%
    None
2. backend/pyspur/nodes/decorator.py:230
  • Draft comment:
    Removed debug logging and print statements to clean up production output. If needed for troubleshooting, consider conditional logging behind a debug flag.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50%
    None

Workflow ID: wflow_Wf4j2g9QywwonUxv


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant