Using the wrap: true option results in the module being wrapped in

(function(angular, undefined) {
    ...
})(angular);

Passing undefined as an argument conflicts with best practices and the no-shadow-restricted-names rule in ESLint. This should not be the default behavior of wrap: true in my opinion.

https://eslint.org/docs/rules/no-shadow-restricted-names

A worka

The WordPress project has switched to using wp i18n make-pot (docs). It'd be great if this package switched over too, for two specific advantages:

1. Full PHP 7 support.
2. Also parses JavaScript files.

cc @schlessera @swissspidy

Hey!

I am trying to use grunt-webdriver to run test suites in sequence instead of running all spec files at once. Here is my conf.js file:

`exports.config = {
//
// ====================
// Runner Configuration
// ====================
//
// WebdriverIO allows it to run your tests in arbitrary locations (e.g. locally or
// on a remote machine).
runner: '

Hi benkeen, great work here. I just spent the last hour adding this to a bunch of our repos, can't wait to find all the bad code with it.

The Problem

My biggest issue is that I have to do some pretty icky regexes to determine if a search match is ok or not. A great example is that in our js code, 90% of the time it is a mistake to include a console.log statement. That usually indicates s