refactor: remove noisy debug logging from codebase#218
Open
SerhiiFesenko wants to merge 2 commits intobluzky:mainfrom
Open
refactor: remove noisy debug logging from codebase#218SerhiiFesenko wants to merge 2 commits intobluzky:mainfrom
SerhiiFesenko wants to merge 2 commits intobluzky:mainfrom
Conversation
Remove IO.inspect calls from core components and development tools to reduce noise in production environments: - Remove dispatch_command debug logging from liveview.ex - Remove file path debug logging from salad.install mix task - Remove JSX conversion debug logging from storybook - Remove form submission debug logging from demo page Keeps user-facing output from mix tasks for installation feedback.
Remove console.log statements from JavaScript components to reduce noise: - Remove client command debug logging from component.js - Remove animation debug logging from utils.js Keeps essential console.error and console.warn statements for production debugging.
|
🧙 Sourcery has finished reviewing your pull request! Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Remove debug logging statements from Elixir and JavaScript code that add noise to production environments. Cleaned up IO.inspect calls and console.log statements while preserving essential error handling.
Summary by Sourcery
Remove noisy debug logging by stripping out IO.inspect and console.log statements from Elixir and JavaScript code
Enhancements: