Show More
@@ -1,157 +1,160 b'' | |||
|
1 | 1 | .. _dev-setup: |
|
2 | 2 | |
|
3 | 3 | =================== |
|
4 | 4 | Development setup |
|
5 | 5 | =================== |
|
6 | 6 | |
|
7 | 7 | |
|
8 | 8 | RhodeCode Enterprise runs inside a Nix managed environment. This ensures build |
|
9 | 9 | environment dependencies are correctly declared and installed during setup. |
|
10 | 10 | It also enables atomic upgrades, rollbacks, and multiple instances of RhodeCode |
|
11 | 11 | Enterprise running with isolation. |
|
12 | 12 | |
|
13 | 13 | To set up RhodeCode Enterprise inside the Nix environment, use the following steps: |
|
14 | 14 | |
|
15 | 15 | |
|
16 | 16 | |
|
17 | 17 | Setup Nix Package Manager |
|
18 | 18 | ------------------------- |
|
19 | 19 | |
|
20 | 20 | To install the Nix Package Manager, please run:: |
|
21 | 21 | |
|
22 | 22 | $ curl https://nixos.org/nix/install | sh |
|
23 | 23 | |
|
24 | 24 | or go to https://nixos.org/nix/ and follow the installation instructions. |
|
25 | 25 | Once this is correctly set up on your system, you should be able to use the |
|
26 | 26 | following commands: |
|
27 | 27 | |
|
28 | 28 | * `nix-env` |
|
29 | 29 | |
|
30 | 30 | * `nix-shell` |
|
31 | 31 | |
|
32 | 32 | |
|
33 | 33 | .. tip:: |
|
34 | 34 | |
|
35 | 35 | Update your channels frequently by running ``nix-channel --upgrade``. |
|
36 | 36 | |
|
37 | 37 | |
|
38 | 38 | Switch nix to the latest STABLE channel |
|
39 | 39 | --------------------------------------- |
|
40 | 40 | |
|
41 | 41 | run:: |
|
42 | 42 | |
|
43 | 43 | nix-channel --add https://nixos.org/channels/nixos-16.03 nixpkgs |
|
44 | 44 | |
|
45 | 45 | Followed by:: |
|
46 | 46 | |
|
47 | 47 | nix-channel --update |
|
48 | 48 | |
|
49 | 49 | |
|
50 | 50 | Clone the required repositories |
|
51 | 51 | ------------------------------- |
|
52 | 52 | |
|
53 | 53 | After Nix is set up, clone the RhodeCode Enterprise Community Edition and |
|
54 | 54 | RhodeCode VCSServer repositories into the same directory. |
|
55 | 55 | To do this, use the following example:: |
|
56 | 56 | |
|
57 | 57 | mkdir rhodecode-develop && cd rhodecode-develop |
|
58 | 58 | hg clone https://code.rhodecode.com/rhodecode-enterprise-ce |
|
59 | 59 | hg clone https://code.rhodecode.com/rhodecode-vcsserver |
|
60 | 60 | |
|
61 | 61 | .. note:: |
|
62 | 62 | |
|
63 | 63 | If you cannot clone the repository, please request read permissions |
|
64 | 64 | via support@rhodecode.com |
|
65 | 65 | |
|
66 | 66 | |
|
67 | 67 | |
|
68 | 68 | Enter the Development Shell |
|
69 | 69 | --------------------------- |
|
70 | 70 | |
|
71 | 71 | The final step is to start the development shell. To do this, run the |
|
72 | 72 | following command from inside the cloned repository:: |
|
73 | 73 | |
|
74 | 74 | cd ~/rhodecode-enterprise-ce |
|
75 | 75 | nix-shell |
|
76 | 76 | |
|
77 | 77 | .. note:: |
|
78 | 78 | |
|
79 | 79 | On the first run, this will take a while to download and optionally compile |
|
80 | 80 | a few things. The following runs will be faster. The development shell works |
|
81 | 81 | fine on both MacOS and Linux platforms. |
|
82 | 82 | |
|
83 | 83 | |
|
84 | 84 | |
|
85 | 85 | Creating a Development Configuration |
|
86 | 86 | ------------------------------------ |
|
87 | 87 | |
|
88 | 88 | To create a development environment for RhodeCode Enterprise, |
|
89 | 89 | use the following steps: |
|
90 | 90 | |
|
91 | 91 | 1. Create a copy of `~/rhodecode-enterprise-ce/configs/development.ini` |
|
92 | 92 | 2. Adjust the configuration settings to your needs |
|
93 | 93 | |
|
94 | 94 | .. note:: |
|
95 | 95 | |
|
96 | 96 | It is recommended to use the name `dev.ini`. |
|
97 | 97 | |
|
98 | 98 | |
|
99 | 99 | Setup the Development Database |
|
100 | 100 | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ |
|
101 | 101 | |
|
102 | 102 | To create a development database, use the following example. This is a one |
|
103 | 103 | time operation:: |
|
104 | 104 | |
|
105 | 105 | paster setup-rhodecode dev.ini \ |
|
106 | 106 | --user=admin --password=secret \ |
|
107 | 107 | --email=admin@example.com \ |
|
108 | 108 | --repos=~/my_dev_repos |
|
109 | 109 | |
|
110 | 110 | |
|
111 | 111 | Compile CSS and JavaScript |
|
112 | 112 | ^^^^^^^^^^^^^^^^^^^^^^^^^^ |
|
113 | 113 | |
|
114 | To use the application's frontend, you will need to compile the CSS and | |
|
115 | JavaScript with Grunt. This is easily done from within the nix-shell using the | |
|
116 | following command:: | |
|
114 | To use the application's frontend and prepare it for production deployment, | |
|
115 | you will need to compile the CSS and JavaScript with Grunt. | |
|
116 | This is easily done from within the nix-shell using the following command:: | |
|
117 | ||
|
118 | grunt | |
|
117 | 119 | |
|
118 | make web-build | |
|
120 | When developing new features you will need to recompile following any | |
|
121 | changes made to the CSS or JavaScript files when developing the code:: | |
|
119 | 122 |
|
|
120 | You will need to recompile following any changes made to the CSS or JavaScript | |
|
121 | files. | |
|
123 | grunt watch | |
|
122 | 124 | |
|
125 | This prepares the development (with comments/whitespace) versions of files. | |
|
123 | 126 | |
|
124 | 127 | Start the Development Server |
|
125 | 128 | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ |
|
126 | 129 | |
|
127 | 130 | From the rhodecode-vcsserver directory, start the development server in another |
|
128 | 131 | nix-shell, using the following command:: |
|
129 | 132 | |
|
130 | 133 | pserve configs/development.ini http_port=9900 |
|
131 | 134 | |
|
132 | 135 | In the adjacent nix-shell which you created for your development server, you may |
|
133 | 136 | now start CE with the following command:: |
|
134 | 137 | |
|
135 | 138 | |
|
136 | 139 | rcserver dev.ini |
|
137 | 140 | |
|
138 | 141 | .. note:: |
|
139 | 142 | |
|
140 | 143 | To automatically refresh - and recompile the frontend assets - when changes |
|
141 | 144 | are made in the source code, you can use the option `--reload`. |
|
142 | 145 | |
|
143 | 146 | |
|
144 | 147 | Run the Environment Tests |
|
145 | 148 | ^^^^^^^^^^^^^^^^^^^^^^^^^ |
|
146 | 149 | |
|
147 | 150 | Please make sure that the tests are passing to verify that your environment is |
|
148 | 151 | set up correctly. RhodeCode uses py.test to run tests. |
|
149 | 152 | While your instance is running, start a new nix-shell and simply run |
|
150 | 153 | ``make test`` to run the basic test suite. |
|
151 | 154 | |
|
152 | 155 | |
|
153 | 156 | Need Help? |
|
154 | 157 | ^^^^^^^^^^ |
|
155 | 158 | |
|
156 | 159 | Join us on Slack via https://rhodecode.com/join or post questions in our |
|
157 | 160 | Community Portal at https://community.rhodecode.com |
@@ -1,177 +1,182 b'' | |||
|
1 | 1 | |
|
2 | 2 | ====================== |
|
3 | 3 | Contribution Standards |
|
4 | 4 | ====================== |
|
5 | 5 | |
|
6 | 6 | Standards help to improve the quality of our product and its development. Herein |
|
7 | 7 | we define our standards for processes and development to maintain consistency |
|
8 | 8 | and function well as a community. It is a work in progress; modifications to this |
|
9 | 9 | document should be discussed and agreed upon by the community. |
|
10 | 10 | |
|
11 | 11 | |
|
12 | 12 | -------------------------------------------------------------------------------- |
|
13 | 13 | |
|
14 | 14 | Code |
|
15 | 15 | ==== |
|
16 | 16 | |
|
17 | 17 | This provides an outline for standards we use in our codebase to keep our code |
|
18 | 18 | easy to read and easy to maintain. Much of our code guidelines are based on the |
|
19 | 19 | book `Clean Code <http://www.pearsonhighered.com/educator/product/Clean-Code-A-Handbook-of-Agile-Software-Craftsmanship/9780132350884.page>`_ |
|
20 | 20 | by Robert Martin. |
|
21 | 21 | |
|
22 | 22 | We maintain a Tech Glossary to provide consistency in terms and symbolic names |
|
23 | 23 | used for items and concepts within the application. This is found in the CE |
|
24 | 24 | project in /docs-internal/glossary.rst |
|
25 | 25 | |
|
26 | 26 | |
|
27 | 27 | Refactoring |
|
28 | 28 | ----------- |
|
29 | 29 | Make it better than you found it! |
|
30 | 30 | |
|
31 | 31 | Our codebase can always use improvement and often benefits from refactoring. |
|
32 | 32 | New code should be refactored as it is being written, and old code should be |
|
33 | 33 | treated with the same care as if it was new. Before doing any refactoring, |
|
34 | 34 | ensure that there is test coverage on the affected code; this will help |
|
35 | 35 | minimize issues. |
|
36 | 36 | |
|
37 | 37 | |
|
38 | 38 | Python |
|
39 | 39 | ------ |
|
40 | 40 | For Python, we use `PEP8 <https://www.python.org/dev/peps/pep-0008/>`_. |
|
41 | 41 | We adjust lines of code to under 80 characters and use 4 spaces for indentation. |
|
42 | 42 | |
|
43 | 43 | |
|
44 | 44 | JavaScript |
|
45 | 45 | ---------- |
|
46 | 46 | This currently remains undefined. Suggestions welcome! |
|
47 | 47 | |
|
48 | However, we have decided to go forward with W3C standards and picked | |
|
49 | WebComponents as the foundation of user interface. New functionality should | |
|
50 | be implemented as components using the | |
|
51 | `Polymer Project` <https://www.polymer-project.org>`_ library | |
|
52 | and should avoid external dependencies like `jquery`. | |
|
48 | 53 | |
|
49 | 54 | HTML |
|
50 | 55 | ---- |
|
51 | 56 | Unfortunately, we currently have no strict HTML standards, but there are a few |
|
52 | 57 | guidelines we do follow. Templates must work in all modern browsers. HTML should |
|
53 | 58 | be clean and easy to read, and additionally should be free of inline CSS or |
|
54 | 59 | JavaScript. It is recommended to use data attributes for JS actions where |
|
55 | 60 | possible in order to separate it from styling and prevent unintentional changes. |
|
56 | 61 | |
|
57 | 62 | |
|
58 | 63 | LESS/CSS |
|
59 | 64 | -------- |
|
60 | 65 | We use LESS for our CSS; see :doc:`frontend` for structure and formatting |
|
61 | 66 | guidelines. |
|
62 | 67 | |
|
63 | 68 | |
|
64 | 69 | Linters |
|
65 | 70 | ------- |
|
66 | 71 | Currently, we have a linter for pull requests which checks code against PEP8. |
|
67 | 72 | We intend to add more in the future as we clarify standards. |
|
68 | 73 | |
|
69 | 74 | |
|
70 | 75 | -------------------------------------------------------------------------------- |
|
71 | 76 | |
|
72 | 77 | Naming Conventions |
|
73 | 78 | ================== |
|
74 | 79 | |
|
75 | 80 | These still need to be defined for naming everything from Python variables to |
|
76 | 81 | HTML classes to files and folders. |
|
77 | 82 | |
|
78 | 83 | |
|
79 | 84 | -------------------------------------------------------------------------------- |
|
80 | 85 | |
|
81 | 86 | Testing |
|
82 | 87 | ======= |
|
83 | 88 | |
|
84 | 89 | Testing is a very important aspect of our process, especially as we are our own |
|
85 | 90 | quality control team. While it is of course unrealistic to hit every potential |
|
86 | 91 | combination, our goal is to cover every line of Python code with a test. |
|
87 | 92 | |
|
88 | 93 | The following is a brief introduction to our test suite. Our tests are primarily |
|
89 | 94 | written using `py.test <http://pytest.org/>`_ |
|
90 | 95 | |
|
91 | 96 | |
|
92 | 97 | Acceptance Tests |
|
93 | 98 | ---------------- |
|
94 | 99 | Also known as "ac tests", these test from the user and business perspective to |
|
95 | 100 | check if the requirements of a feature are met. Scenarios are created at a |
|
96 | 101 | feature's inception and help to define its value. |
|
97 | 102 | |
|
98 | 103 | py.test is used for ac tests; they are located in a repo separate from the |
|
99 | 104 | other tests which follow. Each feature has a .feature file which contains a |
|
100 | 105 | brief description and the scenarios to be tested. |
|
101 | 106 | |
|
102 | 107 | |
|
103 | 108 | Functional Tests |
|
104 | 109 | ---------------- |
|
105 | 110 | These test specific functionality in the application which checks through the |
|
106 | 111 | entire stack. Typically these are user actions or permissions which go through |
|
107 | 112 | the web browser. They are located in rhodecode/tests. |
|
108 | 113 | |
|
109 | 114 | |
|
110 | 115 | Unit Tests |
|
111 | 116 | ---------- |
|
112 | 117 | These test isolated, individual modules to ensure that they function correctly. |
|
113 | 118 | They are located in rhodecode/tests. |
|
114 | 119 | |
|
115 | 120 | |
|
116 | 121 | Integration Tests |
|
117 | 122 | ----------------- |
|
118 | 123 | These are used for testing performance of larger systems than the unit tests. |
|
119 | 124 | They are located in rhodecode/tests. |
|
120 | 125 | |
|
121 | 126 | |
|
122 | 127 | JavaScript Testing |
|
123 | 128 | ------------------ |
|
124 | 129 | Currently, we have not defined how to test our JavaScript code. |
|
125 | 130 | |
|
126 | 131 | |
|
127 | 132 | -------------------------------------------------------------------------------- |
|
128 | 133 | |
|
129 | 134 | Pull Requests |
|
130 | 135 | ============= |
|
131 | 136 | |
|
132 | 137 | Pull requests should generally contain only one thing: a single feature, one bug |
|
133 | 138 | fix, etc.. The commit history should be concise and clean, and the pull request |
|
134 | 139 | should contain the ticket number (also a good idea for the commits themselves) |
|
135 | 140 | to provide context for the reviewer. |
|
136 | 141 | |
|
137 | 142 | See also: :doc:`checklist-pull-request` |
|
138 | 143 | |
|
139 | 144 | |
|
140 | 145 | Reviewers |
|
141 | 146 | --------- |
|
142 | 147 | Each pull request must be approved by at least one member of the RhodeCode |
|
143 | 148 | team. Assignments may be based on expertise or familiarity with a particular |
|
144 | 149 | area of code, or simply availability. Switching up or adding extra community |
|
145 | 150 | members for different pull requests helps to share knowledge as well as provide |
|
146 | 151 | other perspectives. |
|
147 | 152 | |
|
148 | 153 | |
|
149 | 154 | Responsibility |
|
150 | 155 | -------------- |
|
151 | 156 | The community is responsible for maintaining features and this must be taken |
|
152 | 157 | into consideration. External contributions must be held to the same standards |
|
153 | 158 | as internal contributions. |
|
154 | 159 | |
|
155 | 160 | |
|
156 | 161 | Feature Switch |
|
157 | 162 | -------------- |
|
158 | 163 | Experimental and work-in-progress features can be hidden behind one of two |
|
159 | 164 | switches: |
|
160 | 165 | |
|
161 | 166 | * A setting can be added to the Labs page in the Admin section which may allow |
|
162 | 167 | customers to access and toggle additional features. |
|
163 | 168 | * For work-in-progress or other features where customer access is not desired, |
|
164 | 169 | use a custom setting in the .ini file as a trigger. |
|
165 | 170 | |
|
166 | 171 | |
|
167 | 172 | -------------------------------------------------------------------------------- |
|
168 | 173 | |
|
169 | 174 | Tickets |
|
170 | 175 | ======= |
|
171 | 176 | |
|
172 | 177 | Redmine tickets are a crucial part of our development process. Any code added or |
|
173 | 178 | changed in our codebase should have a corresponding ticket to document it. With |
|
174 | 179 | this in mind, it is important that tickets be as clear and concise as possible, |
|
175 | 180 | including what the expected outcome is. |
|
176 | 181 | |
|
177 | 182 | See also: :doc:`checklist-tickets` |
General Comments 0
You need to be logged in to leave comments.
Login now