]> cat aescling's git repositories - mastodon.git/blob - CONTRIBUTING.md
Fix shortname problem in emojify() (regression from #5016) (#5032)
[mastodon.git] / CONTRIBUTING.md
1 CONTRIBUTING
2 ============
3
4 There are three ways in which you can contribute to this repository:
5
6 1. By improving the documentation
7 2. By working on the back-end application
8 3. By working on the front-end application
9
10 Choosing what to work on in a large open source project is not easy. The list of [GitHub issues](https://github.com/tootsuite/mastodon/issues) may provide some ideas, but not every feature request has been greenlit. Likewise, not every change or feature that resolves a personal itch will be merged into the main repository. Some communication ahead of time may be wise. If your addition creates a new feature or setting, or otherwise changes how things work in some substantial way, please remember to submit a correlating pull request to document your changes in the [documentation](http://github.com/tootsuite/documentation).
11
12 Below are the guidelines for working on pull requests:
13
14 ## General
15
16 - 2 spaces indentation
17
18 ## Documentation
19
20 - No spelling mistakes
21 - No orthographic mistakes
22 - No Markdown syntax errors
23
24 ## Requirements
25
26 - Ruby
27 - Node.js
28 - PostgreSQL
29 - Redis
30 - Nginx (optional)
31
32 ## Back-end application
33
34 It is expected that you have a working development environment set up. The development environment includes [rubocop](https://github.com/bbatsov/rubocop), which checks your Ruby code for compliance with our style guide and best practices. Sublime Text, likely like other editors, has a [Rubocop plugin](https://github.com/pderichs/sublime_rubocop) that runs checks on files as you edit them. The codebase also has a test suite.
35
36 * The codebase is not perfect, at the time of writing, but it is expected that you do not introduce new code style violations
37 * The rspec test suite must pass
38 * To the extent that it is possible, verify your changes. In the best case, by adding new tests to the test suite. At the very least, by running the server or console and checking it manually
39 * If you are introducing new strings to the user interface, they must be using localization methods
40
41 If your code has syntax errors that won't let it run, it's a good sign that the pull request isn't ready for submission yet.
42
43 ## Front-end application
44
45 It is expected that you have a working development environment set up (see back-end application section). This project includes an ESLint configuration file, with which you can lint your changes.
46
47 * Avoid grave ESLint violations
48 * Verify that your changes work
49 * If you are introducing new strings, they must be using localization methods
50
51 If the JavaScript or CSS assets won't compile due to a syntax error, it's a good sign that the pull request isn't ready for submission yet.
This page took 0.098231 seconds and 6 git commands to generate.