1 |
On Sun, Jan 19, 2014 at 3:51 PM, Alexander Berntsen <alexander@××××××.net>wrote: |
2 |
|
3 |
> -----BEGIN PGP SIGNED MESSAGE----- |
4 |
> Hash: SHA256 |
5 |
> |
6 |
> On 19/01/14 23:54, Alec Warner wrote: |
7 |
> > I'm very against add a bunch of extra rules that have to be |
8 |
> > enforced by hand. I want to make it easy to contribute, not more |
9 |
> > difficult. If bob can run a tool that tells him all the things |
10 |
> > that are wrong with his patch, that avoids us having like 1/3rd of |
11 |
> > the conversations on list ;) |
12 |
> Feel free to write a tool for this, or to contribute to any of the |
13 |
> numerous linters and/or editor plug-ins. It would be much appreciated. |
14 |
> |
15 |
> |
16 |
I already prefer pylint, and I think it does cover most of what I want. I |
17 |
am working a pylintrc patch. |
18 |
|
19 |
|
20 |
> As for the difficulty of PEP 257... I have higher hopes for Portage |
21 |
> contributors than getting stuck at that. If I write a patch that makes |
22 |
> most of the docstrings follow it, then they can infer 99% of the |
23 |
> "extra rules" by just looking at the other functions and methods. If |
24 |
> they fail to comply, we can just mention it. If the docstring |
25 |
> formatting is the biggest issue with their patch, I doubt they'll have |
26 |
> a hard time fixing it. |
27 |
> |
28 |
|
29 |
I'm not saying its hard, I'm saying it is a giant waste of time for the |
30 |
list to tell people 'hey your docstrings are wrong' when they can just run |
31 |
a tool to do it ;) |
32 |
|
33 |
-A |
34 |
|
35 |
|
36 |
|
37 |
|
38 |
> - -- |
39 |
> Alexander |
40 |
> alexander@××××××.net |
41 |
> http://plaimi.net/~alexander |
42 |
> -----BEGIN PGP SIGNATURE----- |
43 |
> Version: GnuPG v2.0.22 (GNU/Linux) |
44 |
> Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ |
45 |
> |
46 |
> iF4EAREIAAYFAlLcZPQACgkQRtClrXBQc7XizAD/Y/Gxc7N6VkgNFWRgP5lmQ84r |
47 |
> UwSne2xaqJYphY9x1TcBAIRpjBHB580edLz/8zpT14lqhW3oOmeMz0pNMB8ssW5d |
48 |
> =+zp5 |
49 |
> -----END PGP SIGNATURE----- |
50 |
> |
51 |
> |