Skip to content

Remove unnecessary debug print statements#238

Merged
jim-junior merged 1 commit into
open-ug:mainfrom
Nimesh861:remove-debug-logs
May 31, 2026
Merged

Remove unnecessary debug print statements#238
jim-junior merged 1 commit into
open-ug:mainfrom
Nimesh861:remove-debug-logs

Conversation

@Nimesh861

Copy link
Copy Markdown
Contributor

Summary

Removed unnecessary debug fmt.Println statements that were printing vague messages to stdout during runtime.

Changes

  • Removed debug print from message publishing flow
  • Removed debug prints from resource definition creation/update flow
  • Removed raw logs print from driver log streaming
  • Removed debug response-body print from test code

Testing

  • Ran go fmt ./...
  • Ran go test ./...
  • Relevant packages pass
  • Full test suite has unrelated Windows file permission failures in internal/config/initialize, because the tests expect Unix file modes

@jim-junior jim-junior left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice work @Nimesh861

@jim-junior
jim-junior merged commit 427698c into open-ug:main May 31, 2026
4 checks passed
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.

2 participants