Skip to content
This repository has been archived by the owner on Dec 1, 2019. It is now read-only.

Add verbose logging flag #508

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

Add verbose logging flag #508

wants to merge 1 commit into from

Conversation

lucaspiller
Copy link

This adds a new configuration flag verbose which defaults to false. It restricts certain 'verbose' logging of this loader behind this flag.

The rationale behind this change is that this verbose logging can break certain Webpack commands and flows. For example, Webpack has --json flag which is supposed to just print the output as JSON, but when this loader is added it outputs the following:

$ webpack --json

[at-loader] Using typescript@2.4.1 from typescript and "tsconfig.json" from /app/tsconfig.json.


[at-loader] Checking started in a separate process...

[at-loader] Ok, 0.341 sec.
{
  "errors": [],
  "warnings": [],
...

With this change these verbose messages are no longer output, but errors are still output as usual.

@eliot-akira
Copy link

Thanks for this pull request, I was looking for an option to reduce verbose log messages.

@s-panferov Would you (re)consider merging this?

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

2 participants