Skip to content

Introduce new argument --hideOutput to hide log in written successfully#27

Open
mmikirtumov wants to merge 2 commits into
endel:masterfrom
mmikirtumov:hide-output
Open

Introduce new argument --hideOutput to hide log in written successfully#27
mmikirtumov wants to merge 2 commits into
endel:masterfrom
mmikirtumov:hide-output

Conversation

@mmikirtumov

Copy link
Copy Markdown

No description provided.

Comment thread index.js Outdated
const fs = require('fs');

const maxOldSpaceSize = process.env.LIMIT || 10240;
const hideChangeFileOutput = !!process.argv.indexOf('--hideOutput') > -1;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No need for !!

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Thanks, removed

@mmikirtumov

Copy link
Copy Markdown
Author

Hi @endel ,

Is it possible to merge it into master and create new version for this library.

Thanks.

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