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

Added quiet mode #417

Closed
wants to merge 1 commit into from
Closed

Added quiet mode #417

wants to merge 1 commit into from

Conversation

tHBp
Copy link

@tHBp tHBp commented Jul 13, 2016

Added optional flag inside options for suppressing the logging.

Adding such a option only affects the logging, precisely grunt.log.writeln.
It doesn't affect any other thing. Still, the tests fail.

Added optional flag inside options for suppressing the logging.
@avdg
Copy link
Contributor

avdg commented Jul 13, 2016

Failures seems to be the same as in other pr. There were a few numbers of new releases on the side of uglifyjs and it seems that even before the minor increase (so releases 2.6.3 and 2.6.4, and 2.7.0 is current stable) there were 'breaking' changes to the output.

It might be important to get #416 merged quickly.

@avdg
Copy link
Contributor

avdg commented Jul 13, 2016

There is already another patch fixing the test. Feel free to rebase the patch or rework from scratch (since the patch isn't big anyway yet)

@tHBp tHBp closed this Jul 14, 2016
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