Skip to content

Commit 0b5dc8b

Browse files
committed
fix: fix lint errors
1 parent 5d63168 commit 0b5dc8b

File tree

3 files changed

+10
-9
lines changed

3 files changed

+10
-9
lines changed

Gruntfile.js

+6-6
Original file line numberDiff line numberDiff line change
@@ -148,7 +148,7 @@ module.exports = function(grunt) {
148148
grunt.loadNpmTasks('grunt-karma');
149149
grunt.loadNpmTasks('grunt-webpack');
150150
grunt.loadNpmTasks('grunt-text-replace');
151-
grunt.loadNpmTasks("grunt-vows");
151+
grunt.loadNpmTasks('grunt-vows');
152152

153153

154154
var rollbarJsSnippet = fs.readFileSync('dist/rollbar.snippet.js');
@@ -160,9 +160,9 @@ module.exports = function(grunt) {
160160
vows: {
161161
all: {
162162
options: {
163-
reporter: "spec"
163+
reporter: 'spec'
164164
},
165-
src: ["test/server.*.test.js"]
165+
src: ['test/server.*.test.js']
166166
}
167167
},
168168

@@ -175,15 +175,15 @@ module.exports = function(grunt) {
175175
replacements: [
176176
// Main rollbar snippet
177177
{
178-
from: new RegExp('^(.*// Rollbar Snippet)[\n\r]+(.*[\n\r])*(.*// End Rollbar Snippet)', 'm'),
178+
from: new RegExp('^(.*// Rollbar Snippet)[\n\r]+(.*[\n\r])*(.*// End Rollbar Snippet)', 'm'), // eslint-disable-line no-control-regex
179179
to: function(match, index, fullText, captures) {
180180
captures[1] = rollbarJsSnippet;
181181
return captures.join('\n');
182182
}
183183
},
184184
// jQuery rollbar plugin snippet
185185
{
186-
from: new RegExp('^(.*// Rollbar jQuery Snippet)[\n\r]+(.*[\n\r])*(.*// End Rollbar jQuery Snippet)', 'm'),
186+
from: new RegExp('^(.*// Rollbar jQuery Snippet)[\n\r]+(.*[\n\r])*(.*// End Rollbar jQuery Snippet)', 'm'), // eslint-disable-line no-control-regex
187187
to: function(match, index, fullText, captures) {
188188
captures[1] = rollbarjQuerySnippet;
189189
return captures.join('\n');
@@ -207,7 +207,7 @@ module.exports = function(grunt) {
207207
grunt.registerTask('test', ['test-server', 'test-browser']);
208208
grunt.registerTask('release', ['build', 'copyrelease']);
209209

210-
grunt.registerTask('test-server', function(target) {
210+
grunt.registerTask('test-server', function(_target) {
211211
var tasks = ['vows'];
212212
grunt.task.run.apply(grunt.task, tasks);
213213
});

browserstack.browsers.js

+3-2
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ var sortedByBrowser = {};
3939
try {
4040
loadBrowsers(require('./browserstack.browsers.json'));
4141
} catch (e) {
42-
console.error('Could not read ./browserstack.browsers.json. Ignoring.', e);
42+
console.error('Could not read ./browserstack.browsers.json. Ignoring.', e); // eslint-disable-line no-console
4343
}
4444

4545

@@ -109,7 +109,8 @@ function download(username, accessKey, callback) {
109109
// Save the downloaded browsers in memory
110110
loadBrowsers(body);
111111

112-
fs.writeFile('./browserstack.browsers.json', JSON.stringify(body, null, ' '), callback);
112+
// TODO: Fix? fs is not defined here.
113+
// fs.writeFile('./browserstack.browsers.json', JSON.stringify(body, null, ' '), callback);
113114
});
114115
}
115116

karma.conf.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ module.exports = function (config) {
7979
loader: 'eslint-loader',
8080
exclude: [/node_modules/, /vendor/, /lib/, /dist/],
8181
options: {
82-
configFile: path.resolve(__dirname, ".eslintrc")
82+
configFile: path.resolve(__dirname, '.eslintrc')
8383
}
8484
},
8585
{

0 commit comments

Comments
 (0)