English contribution guideline is here!
我々は皆さんの AiForms.SettingsView へ貢献を歓迎します。 我々は、皆さんによる貢献を出来るだけ簡単にできるようにしたいと考えています。 その為に、貢献しようとする人は以下の方針に従うよう、お願いします。
- AiForms.SettingsView に貢献する為には GitHub アカウント が必要です。
- 問題や機能要望には Issue を作ってください(まだ Issue が作られていない場合)。
- Issue や Pull request を送るのに、事前の連絡は必要ありません。
- バグを issue で報告する場合、バグを再現する為の説明、エラーの情報、環境を書いてください。
- Issue のタイトルと本文はできるだけ英語で書いてください(不可能な場合は日本語でも OK です)。
- Issue テンプレート が用意されているので、必要な箇所を切り取って使用してください。
- GitHub でリポジトリの fork を作ってください。
- ローカルにクローンしたあと、
git config user.name yourname
とgit config user.email [email protected]
を設定してください。特に、user.email
は必ず GitHub のアカウントで使用している e-mail アドレスを設定してください。
- コードやドキュメントを AiForms.SettingsView に貢献するにはベースとなるブランチから、トピック・ブランチを作ってください。
- 通常、これ(ベースとなるブランチ)は development ブランチです。
- development ブランチから、トピック・ブランチを作るには:
git branch issue_999 development
してからgit checkout issue_999
で新しいブランチに切り替えます。development ブランチ上で作業するのを避けてください。
- commit は合理的(ロジック単位)に分けてください。また目的と関係のないコードの変更は含めないでください(コードフォーマットの変更、不要コードの削除など)。
- commit メッセージが正しいフォーマットにあることを確認してください。commit メッセージはできるだけ英語でお願いします。
必須なコミット情報はここに(できるだけ英語で)。
上にある最初の列はパッチ/コミットの概要を説明します。
この本文はパッチが無い状態のプログラムの行動、なんでこの行動は問題なのか、どうやってパッチが問題を解決するのかを説明します。
- 変更の為にテストが必要ならそのテストが追加されているよう確認してください。
我々は .NET Foundation のコーディングスタイルを使用します。ただし、1点違いがあります。
We use Allman style braces, where each brace begins on a new line.
A single line statement block can go without braces
我々は、常に 中括弧を使用します、1行の if/for/while/etc
でも { }
を省略しないでください。
例:
// 👎🏽 DO NOT USE
if (source == null)
throw new ArgumentNullException("source");
// 👍🏽 GOOD
if (source == null)
{
throw new ArgumentNullException("source");
}
コード中には、コメントも含めて日本語は使わないでください。
- 自分の fork で、変更をトピック・ブランチに push してください。
- AiForms.SettingsView のリポジトリに pull request を投稿してください。
pull request は、以下のように作成してください。
- タイトルは変更の要約を分かりやすく書いてください。
- 本文には、関連する issue の番号を本文に含めてください。( ref #199 など)
- 本文には、その変更が確認できるコードの場所を明記してください。通常これは サンプルプログラム に追加することを推奨します。
- 機能の追加・修正の場合はReadMe-ja.mdの対応する箇所に加筆・修正してください。
- まだ作業中である場合、タイトルの先頭に [WIP] を付けてください。マージ可能になったら、[WIP] を除去し、コメントでお知らせください。
- 作業中に development ブランチが変更された場合は、コンフリクトを解消してから [WIP] を除去してください。
- その作業を予約する意味で、まず [WIP] の付いた pull request を投稿することを許可します。ただし、長い間活動が見られない場合は、クローズされる場合があります。
このドキュメントは、MMP/CONTRIBUTING.md · sn0w75/MMP と CONTRIBUTING.md · amay077/ Xamarin.Forms.GoogleMaps をベースに作成しました。