]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
selftests: drv-net: clarify linters and frameworks in README
authorJakub Kicinski <kuba@kernel.org>
Tue, 28 Apr 2026 20:33:57 +0000 (13:33 -0700)
committerJakub Kicinski <kuba@kernel.org>
Wed, 29 Apr 2026 23:48:21 +0000 (16:48 -0700)
Minor clarifications in the README:
 - call out what linters we expect to be clean
 - make it clear that by "frameworks" we mean code under lib/
   not just factoring code out in the same file

Signed-off-by: Jakub Kicinski <kuba@kernel.org>
tools/testing/selftests/drivers/net/README.rst

index c8588436c2243f1d10e9727fdcc561a58b0e9132..c6bed9a985bc74ebbe7368e65ad5d2e20337f9f7 100644 (file)
@@ -211,8 +211,8 @@ Avoid libraries and frameworks
 
 Test files should be relatively self contained. The libraries should
 only include very core or non-trivial code.
-It may be tempting to "factor out" the common code, but fight that urge.
-Library code increases the barrier of entry, and complexity in general.
+It may be tempting to "factor out" the common code to lib/py/, but fight that
+urge. Library code increases the barrier of entry, and complexity in general.
 
 Avoid mixing test code and boilerplate
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
@@ -290,6 +290,12 @@ or::
   def test(cfg, mode, protocol):
       pass
 
+Linters
+~~~~~~~
+
+We expect clean ``ruff check`` and ``pylint --disable=R``.
+The code should be clean, avoid disabling pylint warnings explicitly!
+
 Running tests CI-style
 ======================