Removing console.log #1

Closed
wdavidw opened this Issue Sep 19, 2011 · 2 comments

2 participants

@wdavidw

Hi,

Would it be possible to remove the console.log on line 84 or at least make it configurable (eg verbose mode)? I'm parsing file that i dont control (eg the Apache Hadoop configuration files). Either ways, i would be pleased to provide you with a pull request if you wish.

Thanks,
David

@wdavidw wdavidw closed this Sep 20, 2011
@wdavidw wdavidw reopened this Sep 20, 2011
@teknopaul
Owner
@teknopaul teknopaul added a commit that referenced this issue Sep 22, 2011
@teknopaul fix for issue #1 38169fa
@teknopaul teknopaul closed this Sep 22, 2011
@teknopaul
Owner

I fixed it anyway, and npm published it at 0.0.5, npm update should remove the comment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment