Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

JavaScript 1 Day 講習資料の更新 #59

Open
7 tasks
dynamis opened this issue Oct 1, 2019 · 1 comment
Open
7 tasks

JavaScript 1 Day 講習資料の更新 #59

dynamis opened this issue Oct 1, 2019 · 1 comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@dynamis
Copy link
Contributor

dynamis commented Oct 1, 2019

これまで Google Docs にオリジナルが入っていた 1Day JS 資料もこのリポジトリに入れてしまい、リンク集ページも独立させた。

https://tutorial.chirimen.org/js/

1day 講習資料の中身の更新または別に追記する形であると望ましいのではないかと思うものメモ:

  • エディタは業界標準の VS Code を紹介
  • VS Code のオンライン版である codesandbox の紹介
  • サンプルコード集のリポジトリを作って CodeSandbox で試せるようにする
  • async/await に関して簡単な説明をマージ
  • 比較的よく使われる関数の説明を追加
    • fetch
  • MDN などの詳細情報源を追加
  • eslint や prettier を使う (codesandbox ならデフォルト有効)
  • コーディングルールを airbnb + prettier と整合させる
    • formatter は prettier
    • lint はもう eslint じゃなくて csb 同様に percel が react-error-overlay (要するに babel) でエラーチェックしているのと同じ方向で良いか? - と思ったら csb も babel と同時に eslint もちゃんと使ってる。
@dynamis dynamis added enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed labels Oct 1, 2019
@dynamis
Copy link
Contributor Author

dynamis commented Oct 1, 2019

help wanted: codesandbox は自動で eslint を導入しているがそこで使われているルールが不明。
codesandbox に合わせていく方針ならこのチュートリアルやドキュメントも同様のルールでコードを書いていきたいところ。

追記: ちょっと調べた感じ CSB では現状まだ eslint のユーザ設定には未対応で決め打ちでこの辺りで読み込んでいるプラグインのルールが使われてる感じかな。結局 airbnb + prettier + react 関係という感じ?
https://github.com/codesandbox/codesandbox-client/blob/master/package.json#L74
codesandbox/codesandbox-client#1933

だとするとやはり取りあえず airbnb + prettier に従っておいて、CSB が自由に出来るようになったら eslint は CSB のデフォルトに合わせるか、 recommended だけのルール https://eslint.org/docs/rules/ にするとか?

{
  "extends": [
      "eslint:recommended"
  ]
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant