Skip to content

Fix possibly-used-before-assignment pylint errors (CI failing on master for pylint)

Szymon Kulis requested to merge 128_fix_ci into master

Description of the changes

Close #128 (closed)

I believe most of the problems reported by pylint are false positives. It looks like the new check/feature is buggy as there are already 5 issues reported in the last two days: https://github.com/pylint-dev/pylint/issues?q=is%3Aissue+is%3Aopen+possibly-used-before-assignment+

I decided to waive some of them and address others.

Check-list

  • Make sure that a changelog entry is present
  • Label the MR according to the content
  • If this MR has a dependency on another MR, specify the order in which the MRs should be merged
  • If the MR is done, please mark it as such by using the ~"Done - awaits merge" label, else mark it as "Draft: " by adding the prefix to the title
  • Designate a reviewer for this MR once done
  • Update the supported features and limitations here

Reviewer check-list

  • Make sure that the title describes the changes in the merge request
  • Make sure that the description of the changes is relevant and it describes all the changes
  • Make sure that the changelog entry is consistent with the changes. If the first release on top was released, then make sure that a new running release is created incrementing the minor index (the version is in the format major.minor.patch)
Edited by Szymon Kulis

Merge request reports