Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add logging to replace print satement for debugging #148

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

JustOscarJ1
Copy link

Replaces the hardcoded print statement used for debugging with a configurable logging setup. Specifically:

Replaces print(f"DEBUG: path to be checked: {path}") with logger.debug(f"path to be checked: {path}").
Introduces a module-level logger that doesn't propagate, to avoid affecting other parts of the library or user's project.
Maintains the same output format for consistency.
Allows users to easily enable or disable debug output without modifying the source code.

This change improves flexibility for debugging while maintaining the existing functionality.

@nadermx
Copy link
Owner

nadermx commented Jul 8, 2024

We'd have to update the read me too no? To debug what would the command be?

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