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

fix: removing "Tracing..." console logging #853

Closed
wants to merge 1 commit into from

Conversation

genne
Copy link

@genne genne commented Mar 22, 2018

Unnecessary logging that clutters the console and makes it harder to spot the real issues you might have.

For example, we have ~200 modules in our project, meaning we get ~200 lines of log every time the bundles are regenerated, which happens every time you change a file (triggered by gulp.watch).

Unnecessary logging that clutters the console and makes it harder to spot the real issues you might have.

For example, we have ~200 modules in our project, meaning we get ~200 lines of log every time the bundles are regenerated, which happens every time you change a file (triggered by `gulp.watch`).
@CLAassistant
Copy link

CLAassistant commented Mar 22, 2018

CLA assistant check
All committers have signed the CLA.

@zewa666
Copy link
Member

zewa666 commented Mar 22, 2018

I wouldnt remove it as its sometimes quite useful. Instead this logs should be configurable via aurelia.json

@genne
Copy link
Author

genne commented Mar 26, 2018

Ok, I can probably fix that.

Out of curiosity, what is the use case for when you need that logging? Only for debugging purposes?

@zewa666
Copy link
Member

zewa666 commented Mar 26, 2018

Yep solely debugging, like figuring out where it stops or what parts are missing and not properly traced

@JeroenVinke
Copy link
Collaborator

@genne I like the idea of being able to configure the output of au build. Keep us up to date on the progress!

@3cp
Copy link
Member

3cp commented Oct 7, 2018

@JeroenVinke this one can be closed.
Now the log is behind au build --debug or au run --debug.

@JeroenVinke JeroenVinke closed this Oct 8, 2018
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.

5 participants