Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commitbbad821

Browse files
authored
Build: Require strict mode in Node.js scripts via ESLint
So far, only browser-based JS files were required to be in strict mode (in thefunction form). This commit adds such a requirement to Node.js scripts wherethe global form is preferred. All Node.js scripts in sloppy mode wereconverted to strict mode.Closesgh-4499
1 parent4504fc3 commitbbad821

File tree

11 files changed

+24
-14
lines changed

11 files changed

+24
-14
lines changed

‎.eslintrc-node.json

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,5 +10,9 @@
1010
"env": {
1111
"es6":true,
1212
"node":true
13+
},
14+
15+
"rules": {
16+
"strict": ["error","global"]
1317
}
1418
}

‎Gruntfile.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
1-
module.exports=function(grunt){
2-
"use strict";
1+
"use strict";
32

3+
module.exports=function(grunt){
44
functionreadOptionalJSON(filepath){
55
varstripJSONComments=require("strip-json-comments"),
66
data={};

‎build/release.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
"use strict";
2+
13
varfs=require("fs");
24

35
module.exports=function(Release){

‎build/release/cdn.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
"use strict";
2+
13
var
24
fs=require("fs"),
35
shell=require("shelljs"),

‎build/release/dist.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
"use strict";
2+
13
module.exports=function(Release,files,complete){
24

35
var

‎build/tasks/build.js

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,9 @@
44
* and includes/excludes specified modules
55
*/
66

7-
module.exports=function(grunt){
8-
9-
"use strict";
7+
"use strict";
108

9+
module.exports=function(grunt){
1110
varfs=require("fs"),
1211
requirejs=require("requirejs"),
1312
Insight=require("insight"),

‎build/tasks/dist.js

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
1-
module.exports=function(grunt){
2-
3-
"use strict";
1+
"use strict";
42

3+
module.exports=function(grunt){
54
varfs=require("fs"),
65
filename=grunt.option("filename"),
76
distpaths=[

‎build/tasks/node_smoke_tests.js

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
1-
module.exports=(grunt)=>{
2-
3-
"use strict";
1+
"use strict";
42

3+
module.exports=(grunt)=>{
54
constfs=require("fs");
65
constspawnTest=require("./lib/spawn_test.js");
76
consttestsDir="./test/node_smoke_tests/";

‎build/tasks/qunit_fixture.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
"use strict";
2+
13
varfs=require("fs");
24

35
module.exports=function(grunt){

‎build/tasks/sourcemap.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
"use strict";
2+
13
varfs=require("fs");
24

35
module.exports=function(grunt){

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp