👍🎉 First off, thanks for taking the time to contribute! 🎉👍
The following is a set of guidelines for contributing to the Flexberry projects, which are hosted in the Flexberry Organization on GitHub. Please take a moment to review this document in order to make the contribution process easy and effective for everyone involved.
There are a few basic rules that makes Git commit messages more readable on the terminal and GitHub. The most important rule is use the first line as a summary and to keep it short.
According to Pro Git: Chapter 5.2 we use this template:
Short (50 chars or less) summary of changes
More detailed explanatory text, if necessary. Wrap it to about 72
characters or so. In some contexts, the first line is treated as the
subject of an email and the rest of the text as the body. The blank
line separating the summary from the body is critical (unless you omit
the body entirely); tools like rebase can get confused if you run the
two together.
Further paragraphs come after blank lines.
- Bullet points are okay, too
- Typically a hyphen or asterisk is used for the bullet, preceded by a
single space, with blank lines in between, but conventions vary here
We want to keep our commit histories clean where possible.
- master / develop / feature- / hotfixes / release- for branches.
- merging with
--no-ffoption; - remember about
git rebase;
- When you send a pull request, please send it to the
developbranch. - Give a short description in the pull request what you're doing and why.
- Try to stick to the existing coding style.
- Use
.editorconfigfile to ensure consistent line-endings, tabs/spaces, etc., if it provided with the repository (see EditorConfig for details). - Make sure you have added the necessary tests for your changes.
- Please follow the rules contained in Flexberry/javascript-style-guide.
- All public API methods must be documented with jsdoc.
- Make sure your changes comply to our StyleCop rules, which are defined in the Settings.Stylecop file in a repository root.