Skip to content

Commit 6ba6699

Browse files
KuniwakKuniwak
committed
Merge pull request mixi-inc#20 from mixi-inc/improve-2nd
いろいろ修正
2 parents 21e2ee4 + 78a4132 commit 6ba6699

File tree

13 files changed

+341
-164
lines changed

13 files changed

+341
-164
lines changed

‎.eslintrc‎

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,13 @@
33
"node": true
44
},
55
"rules": {
6-
"quotes": [2, "single"],
7-
"no-use-before-define": 0
6+
"//": "single quote に統一",
7+
"quotes": [2, "single", "avoid-escape"],
8+
9+
"//": "関数宣言文以外は hoisting 避けのため禁止",
10+
"no-use-before-define": [2, "nofunc"],
11+
12+
"//": "この形式のキャストはくっついていないと読みづらい",
13+
"space-infix-ops": [2, { "int32Hint": false }]
814
}
915
}

‎README.md‎

Lines changed: 36 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,23 @@ JavaScript 初心者が JS の未来を見据えつつ、
1111

1212

1313

14+
トレーニングの目標
15+
------------------
16+
17+
18+
19+
このトレーニングの目標は、
20+
**モジュールを読み書きできるようになり、**
21+
自分の好きなモジュールを
22+
つくれるようになることです。
23+
24+
25+
26+
トレーニングの前に、セットアップを
27+
終わらせてしまいましょう。
28+
29+
30+
1431
セットアップ
1532
------------
1633

@@ -52,7 +69,7 @@ JavaScriptTraining リポジトリを fork します。
5269
下のコマンドをターミナルで実行してください。
5370

5471
cd JavaScriptTraining
55-
npm run setup
72+
npm install
5673

5774

5875

@@ -1047,7 +1064,7 @@ fetch('/users.json')
10471064

10481065

10491066

1050-
実はこの Promsie という複雑な仕組みを使う理由は、
1067+
実はこの Promise という複雑な仕組みを使う理由は、
10511068

10521069
- 並行非同期処理
10531070
- 直列非同期処理
@@ -1118,8 +1135,6 @@ fetch('/api/foo')
11181135

11191136

11201137

1121-
#### モジュール
1122-
11231138
JavaScript は言語機能としてモジュールの
11241139
仕組みをもっていません。
11251140

@@ -1134,7 +1149,16 @@ JavaScript は言語機能としてモジュールの
11341149

11351150

11361151

1137-
[エコシステムの例](https://github.com/wilmoore/frontend-packagers)
1152+
- bower
1153+
- component
1154+
- jam
1155+
- volo
1156+
- npm with browserify
1157+
- spm
1158+
- jspm
1159+
- duo
1160+
1161+
(source: [wilmoore/frontend-packagers](https://github.com/wilmoore/frontend-packagers))
11381162

11391163

11401164

@@ -1156,9 +1180,14 @@ bower は、JavaScript、HTML、CSSなどを
11561180
自分が作ったモジュールを公開することも
11571181
できます。
11581182

1159-
ただ、JavaScript の場合モジュールの仕組みを
1160-
もたないため、この部分を RequireJS などの
1183+
1184+
1185+
ただ、bower はモジュール読み込みの
1186+
仕組みを提供していません。
1187+
1188+
この部分は RequireJS など、別の
11611189
モジュールシステムに頼ることになります。
1190+
11621191
どのモジュールシステムに対応するかという選択は、
11631192
bower によって読み込まれるパッケージ側に
11641193
裁量（責務）があります。

‎package.json‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
"description": "Training course repository for JavaScript by mixi",
55
"main": "index.js",
66
"scripts": {
7-
"setup": "npm install; (cd public; bower install)",
7+
"postinstall": "(cd public; $(npm bin)/bower install)",
88
"help": "$(npm bin)/gulp help",
99
"serve": "$(npm bin)/gulp serve",
1010
"presentation": "$(npm bin)/reveal-md README.md --theme solarized --separator '^\\n\\n\\n' --verticalSeparator '^\\n\\n'"

‎public/.eslintrc‎

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,14 @@
11
{
22
"env": {
3-
"mocha": true
3+
"mocha": true,
4+
"browser": true,
5+
"jquery": true
46
},
57
"globals": {
68
"expect": false
9+
},
10+
"rules": {
11+
"//": "chai の expect 構文が誤警告になるため無視",
12+
"no-unused-expressions": 0
713
}
814
}

‎public/stage1/.eslintrc‎

Lines changed: 0 additions & 11 deletions
This file was deleted.

‎public/stage2/.eslintrc‎

Lines changed: 0 additions & 11 deletions
This file was deleted.

‎public/stage2/tests.js‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -182,7 +182,7 @@ describe('ステージ2（意図した通りに DOM 要素の属性・テキス
182182
});
183183

184184

185-
it('5 番の要素を jQuery を使って時計回り方向に 10 度回転させる', function() {
185+
it('11 番の要素を jQuery を使って時計回り方向に 10 度回転させる', function() {
186186

187187
// ここにコードを記述してください。
188188
// 変更した DOM 要素は $element 変数に代入してください。
@@ -198,7 +198,7 @@ describe('ステージ2（意図した通りに DOM 要素の属性・テキス
198198
});
199199

200200

201-
it('6 番の要素を jQuery を使って上に 20px 移動させる', function() {
201+
it('12 番の要素を jQuery を使って上に 20px 移動させる', function() {
202202

203203
// ここにコードを記述してください。
204204
// 変更した DOM 要素は $element 変数に代入してください。

‎public/stage3/.eslintrc‎

Lines changed: 0 additions & 11 deletions
This file was deleted.

‎public/stage4/.eslintrc‎

Lines changed: 0 additions & 11 deletions
This file was deleted.

‎public/stage5/.eslintrc‎

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,6 @@
11
{
2-
"env": {
3-
"browser": true,
4-
"mocha": true
5-
},
62
"globals": {
7-
"$": false,
8-
"jQuery": false,
93
"fetch": false,
10-
"Promise": false,
11-
"HTMLCollection": false
4+
"Promise": false
125
}
136
}

0 commit comments

Comments
 (0)