branch: elpa/swift-mode commit 787658fbcbf73640befbf150a0439ed709ae7509 Author: taku0 <mxxouy6x3m_git...@tatapa.org> Commit: taku0 <mxxouy6x3m_git...@tatapa.org>
Update CONTRIBUTING.md --- CONTRIBUTING.md | 28 +++++++++++++++++++++++++--- 1 file changed, 25 insertions(+), 3 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 0759319..47a4cc0 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -9,7 +9,29 @@ repository. Follow the guidelines below to make sure everything goes smoothly. - Check that the issue has not already been reported - Check that the issue has not already been fixed in the latest code - Open an issue with a clear title -- Write as grammatically correct as you can in the description. +- Use concise English. +- Use code as our common language. + - An example is worth a thousand words. For example: + + > It should be indented like this: + ```swift + guard + var x = xx else { + foo() + foo() + } + ``` + > But actually it is indented like this: + ```swift + guard + var x = xx else { + foo() + foo() + } + ``` + - A link to the language specification or other project is also worth a + thousand words. For example, “I want this feature (links to a elisp + function of other project)”. ## Pull requests @@ -17,6 +39,7 @@ repository. Follow the guidelines below to make sure everything goes smoothly. - Follow the coding conventions already in use - Verify Emacs Lisp code with `checkdoc` - Add unit tests whenever possible +- Run `make check` and ensure no errors other than known-bugs. - Open a [pull request](https://help.github.com/articles/using-pull-requests) relating to a single issue. @@ -24,8 +47,7 @@ repository. Follow the guidelines below to make sure everything goes smoothly. ### Naming -- Use a `swift-mode-` prefix for all public names. -- Use a `swift-mode--` prefix for all internal names. +Use a `swift-mode:` prefix. ### Docstrings