Skip to content

Latest commit

 

History

History
385 lines (258 loc) · 14.3 KB

CONTRIBUTING.md

File metadata and controls

385 lines (258 loc) · 14.3 KB

Contribution Guide

この書籍へのコントリビュート方法についてガイドです。

GitBook

この書籍はGitBookで作成されています。 また、文章はMarkdownで書かれおり、Markdownの記法についてはGitBookのヘルプを参照してください。

一部、{{book.console}}やHTMLコメントを使った仕組みも含まれているため、詳しくはこのガイドで紹介します。

Issues

次のIssueを受け付けています。

その他のIssueも歓迎しています。

Pull Request

Pull Requestはいつでも歓迎しています。

受け入れるPull Request

次の種類のPull Requestを受け付けています。 基本的なPull Request(特に細かいもの)は、Issueを立てずにPull Requestを送ってもらって問題ありません。

「このような修正/改善はどうでしょう?」という疑問がある場合は、Issueを立てて相談してください。

  • 誤字の修正
  • サンプルコードやスペルの修正
  • 別の説明方法の提案や修正
  • 文章をわかりやすくするように改善
  • ウェブサイトの改善
  • テストの改善

📝 Note: Pull Requestを受け入れるとあなたの貢献が[Contributorsリストに追加されます。 また、Pull Requestを送った内容はこの書籍のライセンス(MIT and CC BY-NC)が適応されます。 これは、あなたの貢献がこの書籍への努力的な寄付となることを意味しています。

受け入れていないPull Request

修正の送り方

文章の誤字の修正程度なら、直接GitHub上で編集してPull Requestを送ってください。

ローカルで編集して送りたい場合は次の手順を試してください。

  1. Forkする
  2. Branchを作る: git checkout -b my-new-feature
  3. テストする: npm install && npm test
  4. 変更をコミットする: git commit -am 'Add some feature'
  5. Pushする: git push origin my-new-feature
  6. Pull Requestを送る :D

修正の確認方法

この書籍はGitBookで作成されています。 npm startを実行後、http://localhost:4000/へアクセスすることで、GitBookのプレビュー表示ができます。

npm run start
# open http://localhost:4000/

また、Pull Requestを出した際にNetlify上へプレビュー用のサイトが公開されます。 Pull Request下部に表示されるCI Statusからプレビュー用のサイトを見られるため、GitBook上での表示を確認できます。

CI Status

テスト

$ npm test を実行するとコードや文章に対するテストを実行できます。

npm test

$ npm run textlint の文章表現のエラーで疑問がある場合は、とりあえずそのままPull Requestを送ってください。 IssueやPull Requestでやりとりしそのエラーを直すか無視するかを決定します。

テストの種類

このプロジェクトでは次のようなテストが npm test で実行されています。 特定のSuffixを持つファイル名を対象にしているテストも存在しています。

  • GitBookのビルドテスト
  • textlintによる文章のLint
  • ESLintによるコードのLint
  • textlint + ESLintによるMarkdown中のインラインコードブロックのLint
  • Markdown中のインラインコードブロックへのDoctest
  • Mochaによる*-test.jsファイルのユニットテスト
  • *-example.jsがJavaScriptとして実行できるかのテスト
  • *-invalid.jsがJavaScriptとして実行できないかのテスト

Markdown中のインラインコードブロックとは次のようなjs言語指定がされたCodeBlockを示しています。

```js
var foo = "string";
```

Doctest

*-example.jsのJavaScriptファイルとMarkdownのインラインコードブロックを対象にDoctestが実行されます。

次のように// => 値というコメントを書いた部分に対してDoctestが実行されます。

const a = 42;
console.log(42); // => 42

これにより、サンプルコードのコメントに書いた評価結果と実際の出力が一致するかをテストしています。

サポートする書式

評価したい式; // => 期待する評価結果

or

console.log(評価したい式); // => 期待する評価結果

基本的には、console.log(式); // => 期待する評価結果を利用し、 console.logが冗長な場合は 式; // => 期待する評価結果と書いても良い。

Doctestエラーのテスト

Doctestの正常系は実行結果と期待結果が一致することです。 一方、そのコードの実行結果がエラーになることを期待する場合もあります。 エラーを期待する場合は、doctest: 期待するエラー名をHTMLコメントに書きます。

例) 実行結果がReferenceErrorとなることを期待するテスト

条件式という変数が定義されていないためエラーとなる。

<!-- doctest: ReferenceError -->
```js
if (条件式)
    実行する文;
```

個別の行がエラーとなることを期待する場合は、正常系のDoctestでも書けるためそちらを推奨します。

```js
NO_DEFINE++; // => ReferenceError
```

Doctest非同期のテスト

DoctestでPromiseやAsync Functionを使った非同期のテストも書けます。 非同期処理を期待する場合は、doctest:async:タイムアウトミリ秒をHTMLコメントに書きます。

例) 実行結果が1000ミリ秒以内に完了する非同期処理をテストする

<!-- doctest:async:1000 -->
```js
function wait(ms){
    return new Promise((resolve) => {
        setTimeout(() => resolve(ms), ms)
    })
}
wait(1000).then(() => {
    console.log(value); // => 1000 
});
```

Note: vmモジュールの制約からタイムアウト指定の時間が正しく指定させていることが前提となっています。

Doctestの無視

CodeBlockの手前に<!-- doctest:disable -->というHTMLコメントがある場合はDoctestをしません。 Node.jsで実行できないビルドインオブジェクトを使うパターンや例外的なケースに利用できます。

例) 無視するケース

次の例はAPIの説明のための擬似コードであるため無視しています。

<!-- doctest:disable -->
```js
array.map(コールバック関数);
```

別の手法:

ファイル名が*-invalid.jsのコードは実行できないことを検証できます。(エラーになるとテストが通る) これを[include]することでより正確に表現できます。

  • *-invalid.jsがJavaScriptとして実行できないかのテスト

関連

ディレクトリ構造

source 下に章や節ごとにディレクトリを切り、 その下にコード(src/)、テスト(test/)、リソース(img/)などを配置して扱います。

└── source
    └── ch1
        ├── basic
        │   └── README.md
        ├── comments
        │   ├── README.md
        │   └── src
        │       └── html-like-comments-example.js
        └── hello
            ├── README.md
            ├── img
            ├── src
            │   └── hello-world.js
            └── test
                └── hello-world-test.js

コミットメッセージ規約

AngularJSのGit Commit Guidelinesをベースとしています。

以下のような形で

  • 1行目に概要
  • 2行目はから改行
  • 3行目から本文

最後に関連するIssue(任意)を書きます。

feat(ngInclude): add template url parameter to events

The `src` (i.e. the url of the template to load) is now provided to the
`$includeContentRequested`, `$includeContentLoaded` and `$includeContentError`
events.

Closes #8453
Closes #8454
                         scope        commit title

        commit type       /                /      
                \        |                |
                 feat(ngInclude): add template url parameter to events

        body ->  The 'src` (i.e. the url of the template to load) is now provided to the
                 `$includeContentRequested`, `$includeContentLoaded` and `$includeContentError`
                 events.

 referenced  ->  Closes #8453
 issues          Closes #8454

commit type としては次のようなものがあります。

  • feat
    • 新しい機能、章、節の追加など
    • 更新履歴に載るような新しいページを追加
  • fix
    • 意味が変わる修正
    • 更新履歴に載るような修正
  • docs
    • 基本的には使わない
    • README.mdやCONTRIBUTING.mdや本体のプロジェクト全体のドキュメントについて
  • refactor
    • 意味が変わらない修正
    • 更新履歴に載らないような修正
  • style
    • スペースやインデントの調整
    • Lintエラーの修正など
  • perf
    • パフォーマンス改善
  • test
    • テストに関して
  • chore
    • その他
    • typoの修正など

commit typeは、迷ったらとりあえずchoreと書きます。 scopeも省略して問題ないので以下のような形でも問題ありません。

chore: コミットメッセージ

書き方の色々

書き方に関するルールや表記統一について

「次の」 or 「上記の」 or 「下記の」

文章に対する相対的な位置の指し示す場合は、 できるかぎり「次のコード」というように事前に説明する。

読んでいる人がサンプルコードのどこに注目すればいいかが事前にわかるように書く。


次のコードは xxx について説明しています。

code

コードの参照方法

事前に説明した特定のコードを参照したい場合は、 サンプルコードのファイル名を参照に利用する。

[import, example.js](src/example.js)

色々文章...

[example.js](#example.js)では、 ....

コードのコンソールモード

次の変数をコードブロックの前に書くことで、コードがインタラクティブに実行できるコンソールモードとして表示されます。 (ウェブのみ)

{{book.console}}
```js
var interactive = "code";
```

importしたコードにも対応しています。

{{book.console}}
[import, example.js](src/example.js)

ES6 or ES2015

ES2015は正式な名称ですが、ES6も一般によく使われている名称です。 どちらの表記をメインに利用するかは以下のIssueで議論した結果、ES2015という表記をメインとしています。 これから出てくる仕様はES2016、ES2017と年号形式であるためそちらに揃えていこうという形です。

現在のECMAScriptバージョンを参照するとき

この書籍では"最新版のECMAScript"という定義は変更される場合があります。 2017年においてはECMAScript 2017ですが、2018年では異なります。

そのため、現在のECMAScriptバージョンを扱うときは{{book.esversion}}という変数を利用します。

現在のECMAScriptの最新版はECMAScript {{book.esversion}}です。

この変数はbook.jsonに定義されています。

var vs. let/const

基本的にコードではletまたはconstを利用します。 varの機能を説明する場合においてはvarを利用します。