Projekt

Obecné

Profil

Policies » Historie » Verze 13

Jan Pašek, 2021-03-03 16:44

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
# _In case the integrity of the software is seriously broken, the task can be returned to the *issue owner*._ 
25 11 Jan Pašek
26 1 Jan Pašek
h2. RedMine
27 2 Jan Pašek
28 4 Jan Pašek
h2. Code
29 5 Jan Pašek
30
h3. General
31
32
* Code and all comments shall be written in English
33 9 Jan Pašek
* Code review is done for all code changes
34
* Unit tests are done for all business logic parts 
35 5 Jan Pašek
* Python docstrings (bellow function header) shall be created for all functions describing the purpose, inputs and outputs
36
* Avoid using names that are too general or too wordy. Strike a good balance between the two
37
* When using CamelCase names, capitalize all letters of an abbreviation (e.g. HTTPServer)
38
39
h3. Packages
40
41
* Package names should be all lower case
42
* When multiple words are needed, an underscore should separate them
43
* It is usually preferable to stick to 1-word names
44
45
h3. Modules
46
47
* Module 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. Classes
52
53
* Class names should follow the UpperCaseCamelCase convention
54
* Exception classes should end in “Error”
55
56
h3. Global Variables
57
58
* Global variables should be all lowercase
59
* Words in a global variable name should be separated by an underscore
60
61
h3. Instance Variables
62
63
* Instance variable names should be all lower case
64
* Words in an instance variable name should be separated by an underscore
65 10 Jan Pašek
* Non-public instance variables should begin with a double underscore
66
* If a protected attribute is necessary to be used, the variable name shall start with a single underscore
67 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
68
69
h3. Methods
70
71
* Method names should be all lower case
72
* Words in a method name should be separated by an underscore
73
* Non-public method should begin with a single underscore
74
* If a method name needs to be mangled, two underscores may begin its name
75
76
h3. Method Arguments
77
78
* Instance methods should have their first argument named ‘self’.
79
* Class methods should have their first argument named ‘cls’
80
81
h3. Functions
82
83
* Function names should be all lower case
84
* Words in a function name should be separated by an underscore
85
86
h3. Constants
87
88
* Constant names must be fully capitalized
89
* Words in a constant name should be separated by an underscore