Gentoo Archives: gentoo-portage-dev

From: Alec Warner <antarus@g.o>
To: gentoo-portage-dev@l.g.o
Subject: Re: [gentoo-portage-dev] [PATCH 0/3] Initial fetch() refactoring
Date: Sun, 19 Jan 2014 23:53:29
Message-Id: CAAr7Pr-L5hFZA8eYSPSPjsY9h9z0Z-iQaquOhBeO+L0QnFD5HA@mail.gmail.com
In Reply to: Re: [gentoo-portage-dev] [PATCH 0/3] Initial fetch() refactoring by Alexander Berntsen
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 >