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 a verbose and debug mode to parcel for better bug reports #1385

Closed
wants to merge 4 commits into from

Conversation

DeMoorJasper
Copy link
Member

@DeMoorJasper DeMoorJasper commented May 17, 2018

This PR adds 2 extra log modes, one verbose mode and debug mode

This way we can just ask to submit their build (debug) log to a bug report in case it's a private project

It will also help with debugging for anyone looking into the issues

logLevel 4 => Verbose
logLevel 5 => Debug/Log mode (creates a log file per build)

Debug log format: parcel-debug-yyyy-mm-dd@hh:mm:ss.log
Verbose line format: [timestamp]: ...

Closes #993 Closes #1527
Related issue, that might benefit from this debug mode: #1331

@devongovett
Copy link
Member

This would probably be useful, but I'm a little worried about performance of all of these extra function calls when it is not enabled. Did you do any perf testing?

@tmchng
Copy link

tmchng commented Aug 1, 2018

@devongovett Honestly I think right now parcel should focus on getting rid of critical bugs first (like #1331).

This PR will be much needed so that people like myself who come across blocking bugs can help out more easily. Performance can come later, and users like myself can wait.

Parcel is a great tool, and it's frustrating that I now need to go back to webpack again due to random freezes.

@DeMoorJasper DeMoorJasper deleted the feature/debug-mode branch August 2, 2018 00:43
@kjellski
Copy link

kjellski commented Aug 7, 2018

@devongovett I think @tmchng is right. We currently have absolutely no option to provide information on why the tool is breaking our builds. For me it suddenly hangs somewhere and then we're done. No way to find out. Going back to webpack seems the only way out currently...

And that's really sad, honestly... I'd like to help, but I can't.

@DeMoorJasper
Copy link
Member Author

Please move discussion to #1834 or open a new issue

@parcel-bundler parcel-bundler locked and limited conversation to collaborators Aug 7, 2018
@DeMoorJasper
Copy link
Member Author

@kjellski Might be related to sass if you're using that, see #1847

Feel free to hit me up on https://spectrum.chat/parcel if you need any help debugging or fixing the issue

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants