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

Blank regular info messages to keep progress messages running beyond them #6

Merged
merged 2 commits into from
Jan 6, 2015

Conversation

evs-chris
Copy link
Contributor

I have a gobblefile that uses a few single source nodes for several separate transform paths before merging some and doing bizarro things with others. Each time a node finishes processing, it spams the output with the same completion message a few times. This squashes repeat info messages to avoid that.

It also adds blanking to regular info messages so that previous progress messages don't get partially dangled beyond the end of them.

@Rich-Harris
Copy link
Contributor

I think the de-duping is probably better handled within gobble itself, so that other dependants (e.g. grunt-gobble) get the same benefit. Just tried it locally and it seems to work - will raise a PR shortly. +1 to the blanking though - I reckon we should keep this PR minus the lastInfoMessage stuff

@evs-chris evs-chris changed the title Squash repeated info messages? Blank regular info messages to keep progress messages running beyond them Jan 6, 2015
@evs-chris
Copy link
Contributor Author

👍 That's even better.

evs-chris added a commit that referenced this pull request Jan 6, 2015
Blank regular info messages to keep progress messages from running beyond them
@evs-chris evs-chris merged commit a8aab57 into gobblejs:master Jan 6, 2015
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