Merge branch 'update-docs' into develop
TODO: * Add info about code layout, debugging tips, etc. * Add "developing powerline" section to docs (not contributing document) Ref #287
This commit is contained in:
commit
bc7c5b784d
|
@ -0,0 +1,77 @@
|
|||
*****************
|
||||
How to contribute
|
||||
*****************
|
||||
|
||||
So you want to contribute to the Powerline project? Awesome! This document
|
||||
describes the guidelines you should follow when making contributions to the
|
||||
project.
|
||||
|
||||
**Please note that these guidelines aren't mandatory in any way, but your
|
||||
pull request will be merged a lot faster if you follow them.**
|
||||
|
||||
Getting started
|
||||
===============
|
||||
|
||||
* Make sure you have a `GitHub account <https://github.com/signup/free>`_.
|
||||
* Submit an `issue on GitHub
|
||||
<https://github.com/Lokaltog/powerline/issues>`_, assuming one does not
|
||||
already exist.
|
||||
|
||||
* Clearly describe the issue.
|
||||
* If the issue is a bug: Make sure you include steps to reproduce, and
|
||||
include the earliest revision that you know has the issue.
|
||||
|
||||
* Fork the repository on GitHub.
|
||||
|
||||
Making changes
|
||||
==============
|
||||
|
||||
* Create a topic branch from where you want to base your work.
|
||||
|
||||
* Powerline uses the `Git Flow
|
||||
<http://nvie.com/posts/a-successful-git-branching-model/>`_ branching
|
||||
model.
|
||||
* Most contributions should be based off the ``develop`` branch.
|
||||
* Prefix your branch with ``feature/`` if you're working on a new feature.
|
||||
* Include the issue number in your topic branch, e.g.
|
||||
``321-fix-some-error`` or ``feature/123-a-cool-feature``.
|
||||
|
||||
* Make commits of logical units.
|
||||
* Run your code through ``flake8`` and fix any programming style errors. Use
|
||||
common sense regarding whitespace warnings, not all warnings need to be
|
||||
fixed.
|
||||
* Make sure your commit messages are in the `proper format
|
||||
<http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html>`_.
|
||||
The summary must be no longer than 70 characters. Refer to any related
|
||||
issues with e.g. ``Ref #123`` or ``Fixes #234`` at the bottom of the
|
||||
commit message. Commit messages can use Markdown with the following
|
||||
exceptions:
|
||||
|
||||
* No HTML extensions.
|
||||
* Only indented code blocks (no ``````` blocks).
|
||||
* Long links should be moved to the bottom if they make the text wrap or
|
||||
extend past 72 columns.
|
||||
|
||||
* Make sure you have added the necessary tests for your changes.
|
||||
* Run *all* the tests to assure nothing else was accidentally broken.
|
||||
|
||||
Programming style
|
||||
-----------------
|
||||
|
||||
* The project uses *tabs for indentation* and *spaces for alignment*, this
|
||||
is also included in a vim modeline on top of every script file.
|
||||
* Run your code through ``flake8
|
||||
--ignore=W191,E501,E121,E122,E123,E128,E225`` to fix any style errors. Use
|
||||
common sense regarding whitespace warnings, not all ``flake8`` warnings
|
||||
need to be fixed.
|
||||
* Trailing whitespace to indicate a continuing paragraph is OK in comments,
|
||||
documentation and commit messages.
|
||||
|
||||
Submitting changes
|
||||
==================
|
||||
|
||||
* Push your changes to a topic branch in your fork of the repository.
|
||||
* If necessary, use ``git rebase -i <revision>`` to squash or reword commits
|
||||
before submitting a pull request.
|
||||
* Submit a pull request to `Lokaltog's repository
|
||||
<https://github.com/Lokaltog/powerline>`_.
|
|
@ -11,35 +11,19 @@ Powerline is licensed under the `MIT license
|
|||
Credits
|
||||
=======
|
||||
|
||||
:Authors:
|
||||
* `Kim Silkebækken <https://github.com/Lokaltog>`_
|
||||
* `ZyX-I <https://github.com/ZyX-I>`_
|
||||
:Main contributors:
|
||||
* `Liam Curry <https://github.com/liamcurry>`_
|
||||
Authors
|
||||
-------
|
||||
|
||||
The glyphs in the font patcher are created by Fabrizio Schiavi, creator of
|
||||
the excellent coding font `Pragmata Pro`_.
|
||||
* `Kim Silkebækken <https://github.com/Lokaltog>`_
|
||||
* `ZyX-I <https://github.com/ZyX-I>`_
|
||||
* `Kovid Goyal <https://github.com/kovidgoyal>`_
|
||||
|
||||
Contributors
|
||||
------------
|
||||
|
||||
* `List of contributors
|
||||
<https://github.com/Lokaltog/powerline/contributors>`_
|
||||
* The glyphs in the font patcher are created by Fabrizio Schiavi, creator of
|
||||
the excellent coding font `Pragmata Pro`_.
|
||||
|
||||
.. _`Pragmata Pro`: http://www.fsd.it/fonts/pragmatapro.htm
|
||||
|
||||
Contributing
|
||||
============
|
||||
|
||||
If you experience any bugs or have any feature requests, please `open an
|
||||
issue on GitHub <https://github.com/Lokaltog/powerline/issues>`_.
|
||||
|
||||
Pull request guidelines
|
||||
=======================
|
||||
|
||||
This project uses `Git Flow`_ for maintaining a clean history and
|
||||
a consistent way of branching and merging new features. All commit messages
|
||||
must follow the guidelines described in `Tim Pope's blog post about git
|
||||
commit messages`_.
|
||||
|
||||
All code must use tabs for indentation and spaces for alignment.
|
||||
|
||||
Python code should pass flake8 tests with ``flake8
|
||||
--ignore=W191,E501,E121,E122,E123,E128``.
|
||||
|
||||
.. _`Git Flow`: http://nvie.com/posts/a-successful-git-branching-model/
|
||||
.. _`Tim Pope's blog post about git commit messages`: http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html
|
||||
|
|
Loading…
Reference in New Issue