public inbox for gentoo-portage-dev@lists.gentoo.org
 help / color / mirror / Atom feed
* [gentoo-portage-dev] [PATCH] First draft of a portage pylint.
@ 2014-01-10  9:38 antarus
  2014-01-11  2:51 ` Alexander Berntsen
  2014-01-13 23:27 ` Mike Frysinger
  0 siblings, 2 replies; 3+ messages in thread
From: antarus @ 2014-01-10  9:38 UTC (permalink / raw
  To: gentoo-portage-dev

From: Alec Warner <antarus@gentoo.org>

---
 pylintrc | 31 +++++++++++++++++++++++++++++++
 1 file changed, 31 insertions(+)
 create mode 100644 pylintrc

diff --git a/pylintrc b/pylintrc
new file mode 100644
index 0000000..cc15ec4
--- /dev/null
+++ b/pylintrc
@@ -0,0 +1,31 @@
+[MASTER]
+profile=no
+ignore=.git
+persistent=no
+
+[REPORTS]
+# Don't print reports by default, they are spammy. We primarily want linting.
+reports=no
+
+[BASIC]
+# Portage uses stuff like i,j,k for counters, e,ex for exceptions.
+# Don't feel too guilty about adding stuff to here.
+
+# NOTE(antarus): Don't add 'x' here, as it is currently used inappropriately
+# in a ton of legacy code. We want to fix those, then I'd consider adding 'x' and 'y'
+good-names=i,j,k,e,ex
+
+[FORMAT]
+# Portage uses tabs for indenting
+indent-string='\t'
+
+# We have arbitrarily chosen 80 chars.
+# Some lines will be anonying to fix (mostly urls, and other long strings.)
+# You should use inline pylint disable messages to silence those.
+max-line-length=80
+
+# Just make it really large for now.
+max-module-lines=20000
+
+#[MESSAGES CONTROL]
+#disable-msg=W0312
-- 
1.8.1.2



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [gentoo-portage-dev] [PATCH] First draft of a portage pylint.
  2014-01-10  9:38 [gentoo-portage-dev] [PATCH] First draft of a portage pylint antarus
@ 2014-01-11  2:51 ` Alexander Berntsen
  2014-01-13 23:27 ` Mike Frysinger
  1 sibling, 0 replies; 3+ messages in thread
From: Alexander Berntsen @ 2014-01-11  2:51 UTC (permalink / raw
  To: gentoo-portage-dev

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

On 10/01/14 10:38, antarus@gentoo.org wrote:
> indent-string='\t'
What does this do with respect to mixing tabs and spaces for
formatting? With PEP-8 this is of course not a problem, since it
requires spaces.

> max-module-lines=20000
I think this is terrible, but given the status of our current modules...
I can sympathise with admitting defeat.


Having this is a good idea, for the record. Thanks for the effort.
- -- 
Alexander
alexander@plaimi.net
http://plaimi.net/~alexander
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.22 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iF4EAREIAAYFAlLQscYACgkQRtClrXBQc7Ut6wD/aIfDCWNg18oZwwrwliwzIlxB
8HDkBMJAuJoOgL+uGFwA/1Qf2lfknURrOBiW+aoryLervYcBET2f4f4VV/NQjkiz
=TuOS
-----END PGP SIGNATURE-----


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [gentoo-portage-dev] [PATCH] First draft of a portage pylint.
  2014-01-10  9:38 [gentoo-portage-dev] [PATCH] First draft of a portage pylint antarus
  2014-01-11  2:51 ` Alexander Berntsen
@ 2014-01-13 23:27 ` Mike Frysinger
  1 sibling, 0 replies; 3+ messages in thread
From: Mike Frysinger @ 2014-01-13 23:27 UTC (permalink / raw
  To: gentoo-portage-dev; +Cc: antarus

[-- Attachment #1: Type: Text/Plain, Size: 184 bytes --]

people have been using pyflakes of late.  not that i'm against adding this.

we should have a wrapper script:
$ cat pylint
#!/bin/sh
exec pylint --rcfile "${0%/*}"/pylintrc "$@"
-mike

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2014-01-13 23:27 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-01-10  9:38 [gentoo-portage-dev] [PATCH] First draft of a portage pylint antarus
2014-01-11  2:51 ` Alexander Berntsen
2014-01-13 23:27 ` Mike Frysinger

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox