* cf. https://bugs.llvm.org/show_bug.cgi?id=37272

Depends-On: 24735
Change-Id: I41ba1142a61a94977b3471baf9415ce1c4fb2df0

Specifying exception names in the overgeneral-exceptions option without
module name is deprecated and support for it will be removed in pylint
3.0.

Change-Id: Ie97cab71718ece3d769a1cc9dc827c7f8a7d0a88

Change-Id: I64074f9573b0ca5d310b81801cebb87312f21529

Change-Id: I599f364f7ecbfe4038e238b23cf2ce4ea430180d

- remove depreacted options in pylintrc
- checking for no-self-use is now optional -> add comment in pylintrc.
  Currently we can not enable the optional check due to a too old
  version in the ASIC container

Depends-On: topic:spack_update_0.20.0
In-Container: /containers/testing/c21398p64_2023-11-03_1.img

Change-Id: I727f447db2c8d635915d33fd3ea95081b81654a1

* includes namespaces in sorting
* leads to better-readable order

Change-Id: I04f68e49d391457da7fb8782611e0b4ed13aa714

Change-Id: I10c3db857bd2ab25fb447e1988e6a2509f79ef4f

Change-Id: I5f5f1cb5bc4a406355f51d98082eb84cd89a3759

Change-Id: Ia6a9d36552d07ca91deb9d83db2e3c5a46cfa8d0

Change-Id: Id9a960e217a0360ec89dd093020988312d6aa17f

* gets rid of some warnings

Change-Id: Icde5a02f403feeaf50aaf533c2ca143d2231addb

Pylint's 'duplicate-code' warning cannot be disabled on a
per-snippet/file basis and therefore needs to be disabled globally to
cope with valid usages of multiple similar lines.

Upstream issue: https://github.com/PyCQA/pylint/issues/214

Change-Id: I0693406c0a48fb7029b6ca18cc01852839a48cbc

* orders non-system headers first and system headers below.
* this allows early fail for non-existant system header dependencies
  in included non-system headers

Change-Id: Ieaba7dd85232f5a2fc4d8244c0938a318406dae4

Some variable names are so common, that it is more confusing not to use
them, although they are short.

This change adds `fd` and `ax`.

Change-Id: Ibe9b8425df106d51fffe2e2ecad81a5a943c801e

Change-Id: Ie28e41982fbb843c1e65086c5e057f2c73d914c9

Change-Id: I57010c6d62f82407921d6433d5744f893cc0857f

Check for blacklisted names ‘l’, ‘O’, and ‘I’ with pylint instead of
pycodestyle, this allows to disable the check inline.

Change-Id: I765f0d62d541fe55125396ff55ee3597f8086987

The one-true-source-of-visionary-coding-style™ has stated that inline
functions are _not_ visionary style.

https://gerrit.bioai.eu/c/hxcomm/+/10315/38/include/hxcomm/common/stream.h#64

We have to adjust the coding style accordingly.

Change-Id: Ie0602c3eebf99faffaf5ec508b294bf76d4af2e9

In order minimize clashes of changes in clang-format, force their
ordering to be alphabetically.

Change-Id: I284c05b80bc92893f7d8ed120a2edc4636182e6e

In order to be convertable to sphinx, the filenames of the documentation
need to be lowercase to match the lowercase ids used in sphinx.

Change-Id: Ie28c08313c44665590f8bc354cde9a94499d47ea

The xml output can be used by other tools to create different formats,
e.g. for sphinx based documentation.

Change-Id: I4419bfbe8d9dc752e212edbc2cf635d5e5ebb59b

Change-Id: I73ea403626374236d6e41ec711e8b1f3af66b691

Change-Id: I1d30413d5e69507b57d8d685762167ce87cfcde3

Currently, the length of a line is checked with pycodestyle, which does
not take into account pylint comments, e.g. disabling an error for one
line. Thus, errors can only be disabled per code block, as otherwise
lines would easily get too long. Pylint itself ignores the line length
error if it is caused by such comments.

Depends-On: 9326

Change-Id: Ib231e73fe73ddff7d9163afe607e7a26b95e6305

Fix for https://github.com/pytorch/pytorch/issues/701

Change-Id: I3a652c7de7b4756fa0421c14282928994155ca28

Fix linter behaviour concerning issue 3449.

Change-Id: I7ad040419d3acb21244b4072d8ea70c2d0c596d2

Increase number of lines that have to be similar to get a duplicate code
warning from 4 to 10. The warning triggers frequently on function
headers or longer configs which are partly similar.

Change-Id: I9a3a2d70a62332aef5d5d36f3655c75f2e0efa54

* get's rid of all the informational stdout printout except
  warnings and errors

Change-Id: I25c085342508e7cf4d24443bb27b8f6f95b707f3

As discussed here: https://openproject.bioai.eu/meetings/237

Change-Id: I7a362f7a38ce3730ebfbcadbb62323bd78dc6b44

* with this option enabled, dot is called with -Tpdf which is not
  supported by the current build options in the graphviz spack
  package

Change-Id: I8e755d3bc86ca00585bd7f96bba7a456ec568779

Yannik Stradmann authored 5 years ago and Yannik Stradmann committed 5 years ago

* W503 is in conflict with the more modern W504
  See https://github.com/PyCQA/pycodestyle/pull/502

Change-Id: I1f29b362e0f0c6893a81c5cd12a862890e8d8609

Change-Id: Icd659e6224ff3968d90bfe6ab074c284423d7de4

* Enables usage of c++17 features like 'if constexpr'

Change-Id: I006bdc725c62a56e8040d1f207a76d85c71df037

Change-Id: I66cdb59ae68d285811526eaabbe815e838ace8c9

Change-Id: I55a9dcb1cce3e8182764a10e6ea5a26414318ac2

Change-Id: Id41f205e53d0b52e28c75861d110eded6f0c33eb

Change-Id: I33372e0766616625be58851be81311010382f863

Yannik Stradmann authored 6 years ago and Yannik Stradmann committed 6 years ago

This is a fresh start for visionary python linting. I tried to find a
reasonably "hard" default configuration without too much noise. Legacy
projects with imperfect code quality might need project-specific less
restrictive rules. New code should fulfill high standards.

Change-Id: I5e4fbe4c818db2dc95e1215dd84fb2edcf659d88

Change-Id: I76980baab8214ba158c53937ed48e2e6a29d607c

Change-Id: I5b71fd82dbd0527c4d6f5b09d51b87a3be9d249d