Couto / groundskeeper

node.js package to remove forgotten console statements and specific blocks of code from you Javascript files

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

namespace options strips out entire files

pbakaus opened this issue · comments

Unfortunately, if I call groundskeeper with namespace set to "s5.Log" on the following file:

core.Class('boiler.App', {
    include: [],

    construct: function (settings) {
        s5.Log.test = true;
    }
});

it will strip out the entire file, as opposed to just the line.

Probably related, groundskeeper just errors out on this file:

function foo() {

    function bar() {
        s5.Log.test = true;
    };

};

any help is greatly appreciated!

Im currently away from my house, so I can't access a computer right now, I will take a look at this over the weekend.

Thanks for reporting.

Looking forward to it, thanks for the quick response! I think this is related to the assignment (function calls seem to work fine, vs. assignments).

This is passing in my tests now and I've updated groundskeeper in npm.
Please report anymore bug you can find please. =)

Awesome, great job, thanks so much for your quick help! Will try on Monday!