diff options
author | Dave Andreoli <dave@gurumeditation.it> | 2018-08-26 11:42:24 +0200 |
---|---|---|
committer | Dave Andreoli <dave@gurumeditation.it> | 2018-08-26 11:42:24 +0200 |
commit | 42ee207c166dba96ccb8ffdfb9cd035e573de80c (patch) | |
tree | fb090a0db7315042667893d348424aa255f0ab3c /CODING | |
parent | c9f5b53e281834ad1f531f63e2743c0b4bc36e82 (diff) |
In development mode again
Diffstat (limited to 'CODING')
-rw-r--r-- | CODING | 29 |
1 files changed, 1 insertions, 28 deletions
@@ -105,7 +105,6 @@ Release process instructions | |||
105 | * Update the ChangeLog file: | 105 | * Update the ChangeLog file: |
106 | setup.py build_doc -b changes ...and manually merge from the html file | 106 | setup.py build_doc -b changes ...and manually merge from the html file |
107 | * Git push and wait jenkins to generate the tarballs and the documentation | 107 | * Git push and wait jenkins to generate the tarballs and the documentation |
108 | * HACK: Use cython 23.5 to generate the source and the tarball on your pc !! see T5589 | ||
109 | * Test the generated tarballs | 108 | * Test the generated tarballs |
110 | * scp tarballs & md5sums to: | 109 | * scp tarballs & md5sums to: |
111 | download.enlightenment.org:/srv/web/download.enlightenment.org/public_html/pre-releases/ | 110 | download.enlightenment.org:/srv/web/download.enlightenment.org/public_html/pre-releases/ |
@@ -136,31 +135,5 @@ Release process instructions | |||
136 | 135 | ||
137 | more info at: | 136 | more info at: |
138 | phab.enlightenment.org/w/release_procedure/ | 137 | phab.enlightenment.org/w/release_procedure/ |
138 | phab.enlightenment.org/w/hosting/ssh/ | ||
139 | 139 | ||
140 | |||
141 | Discussion | ||
142 | ========== | ||
143 | |||
144 | * Internal utility functions used in the bindings must start with an | ||
145 | underscore and must have the shortest name as possible. | ||
146 | ^ | ||
147 | This needs further discussion/expansion. | ||
148 | |||
149 | When we define a function with cdef it is not exposed to Python API. | ||
150 | This should be explicit enough to not need the underscore prefix, which | ||
151 | at best looks ugly, and at worst just plain confusing. | ||
152 | |||
153 | A function name should summarize its functionality in one clear text, | ||
154 | short sentence. We have both too long and too short names. And I admit to | ||
155 | being guilty of adding many of both. | ||
156 | |||
157 | Let's build up a short review so we can see where we stand with this and | ||
158 | make necessary corrections. | ||
159 | |||
160 | / kuuko | ||
161 | |||
162 | |||
163 | The underscore usage is a coding standard in all the EFL, we should try | ||
164 | to follow the efl style also here (where is possible and make sense) | ||
165 | |||
166 | / davemds | ||