Policies » Historie » Verze 14
Jan Pašek, 2021-03-03 16:48
1 | 3 | Jan Pašek | h1. Project convention |
---|---|---|---|
2 | 1 | Jan Pašek | |
3 | h2. Git |
||
4 | 6 | Jan Pašek | |
5 | 13 | Jan Pašek | *Commits* |
6 | |||
7 | 12 | Jan Pašek | Commit message shall have the following format: |
8 | <pre> |
||
9 | #<id> - <short description> <details> |
||
10 | </pre> |
||
11 | |||
12 | * *id* issue ID from the Redmine |
||
13 | * *short description* brief description of the change |
||
14 | * *details* detailed list of changes (added classes, changed interfaces, ...) |
||
15 | 13 | Jan Pašek | |
16 | Each commit shall have a reasonable size to make the reviews as simple as possible. |
||
17 | |||
18 | *Branches* |
||
19 | |||
20 | # For each issue, a new branch shall be created. The branch name shall correlate with the issue name. |
||
21 | # When the work on a branch is finished by the *issue owner*, the work is posted for a code review. |
||
22 | # After that, the code can be merged to master by the *reviewer*. |
||
23 | # After the merge, the *reviewer* is responsible for verifying software integrity. |
||
24 | 14 | Jan Pašek | # _In case the integrity of the software is seriously broken, the task can be returned to the *issue owner*._ |
25 | |||
26 | *Code review* |
||
27 | |||
28 | * Reviewer walks through the code and checks the coding convention, code readability, stability, etc... |
||
29 | * Reviewer is responsible for basic functionality testing |
||
30 | |||
31 | 11 | Jan Pašek | |
32 | 1 | Jan Pašek | h2. RedMine |
33 | 2 | Jan Pašek | |
34 | 4 | Jan Pašek | h2. Code |
35 | 5 | Jan Pašek | |
36 | h3. General |
||
37 | |||
38 | * Code and all comments shall be written in English |
||
39 | 9 | Jan Pašek | * Code review is done for all code changes |
40 | * Unit tests are done for all business logic parts |
||
41 | 5 | Jan Pašek | * Python docstrings (bellow function header) shall be created for all functions describing the purpose, inputs and outputs |
42 | * Avoid using names that are too general or too wordy. Strike a good balance between the two |
||
43 | * When using CamelCase names, capitalize all letters of an abbreviation (e.g. HTTPServer) |
||
44 | |||
45 | h3. Packages |
||
46 | |||
47 | * Package names should be all lower case |
||
48 | * When multiple words are needed, an underscore should separate them |
||
49 | * It is usually preferable to stick to 1-word names |
||
50 | |||
51 | h3. Modules |
||
52 | |||
53 | * Module names should be all lower case |
||
54 | * When multiple words are needed, an underscore should separate them |
||
55 | * It is usually preferable to stick to 1 word names |
||
56 | |||
57 | h3. Classes |
||
58 | |||
59 | * Class names should follow the UpperCaseCamelCase convention |
||
60 | * Exception classes should end in “Error” |
||
61 | |||
62 | h3. Global Variables |
||
63 | |||
64 | * Global variables should be all lowercase |
||
65 | * Words in a global variable name should be separated by an underscore |
||
66 | |||
67 | h3. Instance Variables |
||
68 | |||
69 | * Instance variable names should be all lower case |
||
70 | * Words in an instance variable name should be separated by an underscore |
||
71 | 10 | Jan Pašek | * Non-public instance variables should begin with a double underscore |
72 | * If a protected attribute is necessary to be used, the variable name shall start with a single underscore |
||
73 | 5 | Jan Pašek | * If an instance name needs to be mangled (interpreter rewrites the name in order to avoid name conflicts in subclasses), two underscores may begin its name |
74 | |||
75 | h3. Methods |
||
76 | |||
77 | * Method names should be all lower case |
||
78 | * Words in a method name should be separated by an underscore |
||
79 | * Non-public method should begin with a single underscore |
||
80 | * If a method name needs to be mangled, two underscores may begin its name |
||
81 | |||
82 | h3. Method Arguments |
||
83 | |||
84 | * Instance methods should have their first argument named ‘self’. |
||
85 | * Class methods should have their first argument named ‘cls’ |
||
86 | |||
87 | h3. Functions |
||
88 | |||
89 | * Function names should be all lower case |
||
90 | * Words in a function name should be separated by an underscore |
||
91 | |||
92 | h3. Constants |
||
93 | |||
94 | * Constant names must be fully capitalized |
||
95 | * Words in a constant name should be separated by an underscore |