-
Notifications
You must be signed in to change notification settings - Fork 71
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 additional logging for turbosnap #758
Conversation
@ethriel3695 can you share some examples of what the logs look like before and after? The things I would like to see added are:
|
…ging-for-dependent-story-files
Should it be logging the same normalized paths so many times? |
I've tried a few different options for logging the normalized path, and honestly, I don't love any of the iterations. What do you think about removing that from the debug log? |
I think it's better not to have it than to spam it like that. |
remove excesive logs
@tmeasday I removed the excessive logs. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM lets get this merged
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM lets get this merged
I added some debug logging for the traced Files and affected Modules to assist with debugging CI configurations where we are not able to trace files, even with
storybookBaseDir
andstorybookConfigDir