Skip to content
This repository has been archived by the owner on Sep 5, 2023. It is now read-only.

pylint: disable pylint in failing EOL OSes #1329

Closed
ldorau opened this issue Nov 18, 2021 · 1 comment
Closed

pylint: disable pylint in failing EOL OSes #1329

ldorau opened this issue Nov 18, 2021 · 1 comment
Assignees
@osalyk
Labels
build system enhancement New feature or request

Comments

@ldorau
Copy link
Member

ldorau commented Nov 18, 2021 •
edited
Loading

  1. Disable pylint in failing EOL OSes:
  2. Turn on the "len-as-condition" pylint check

See: #1319

Error:
/rpma/tools/perf/lib/figure.py:300:19: C1801: Do not use len(SEQUENCE) to determine if a sequence is empty (len-as-condition)

Failing OSes:
https://github.com/ldorau/rpma/runs/4239593859

This is a bug in Pylint (pylint-dev/pylint#2684), fixed as of Pylint 2.4 (pylint-dev/pylint#2815)
@ldorau ldorau added enhancement New feature or request build system labels Nov 18, 2021
@osalyk
Copy link
Contributor

osalyk commented Aug 25, 2022

Currently, all tests are passing: https://github.com/ldorau/rpma/runs/8012947216?check_suite_focus=true

@osalyk osalyk closed this as completed Aug 25, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@osalyk osalyk

Labels
build system enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ldorau @osalyk