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

Allow muting log messages #467

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Allow muting log messages #467

wants to merge 2 commits into from

Conversation

trkoch
Copy link

@trkoch trkoch commented Oct 23, 2015

Mute output by providing task level option silently: true. When grunt
runs in verbose mode (e.g. --verbose command line option) output is
always logged no matter if the option is set.

Cleans up console output considerably, especially when providing a separate
watch target for live reloading.

Mute output by providing task level option `silently: true`. When grunt
runs in verbose mode (e.g. `--verbose` command line option) output is
always logged no matter if the option is set.
@trkoch
Copy link
Author

trkoch commented Oct 23, 2015

I'd like to make this configurable on the target level. Any ideas?

Alias silent and silently. Both styles are common in the Grunt plugin
ecosystem.
@trkoch
Copy link
Author

trkoch commented Oct 27, 2015

Example output after changing a *.scss file.

screenshot

#### options.silent
Type: `Boolean`
Default: `false`
Aliases: `silently`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

silent i think is more standard-ish

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Came to same conclusion. See second commit. I provided an alias though for better consistency, in case silently is used elsewhere (e.g. when using grunt-contrib-connect). If you consider this feature creep lets remove the alias though.

@trkoch
Copy link
Author

trkoch commented Feb 25, 2016

How can we get this into master?

Base automatically changed from master to main March 22, 2021 15:01
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