Skip to content

pylint: enable pylint#13

Open
alesax wants to merge 13 commits into
linux-nvme:masterfrom
alesax:enable-pylint
Open

pylint: enable pylint#13
alesax wants to merge 13 commits into
linux-nvme:masterfrom
alesax:enable-pylint

Conversation

@alesax

@alesax alesax commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Let's enable pylint checks to ensure better quality of code.

alesax and others added 13 commits July 9, 2026 14:25
Signed-off-by: Ales Novak <alnovak@suse.com>
Signed-off-by: Ales Novak <alnovak@suse.com>
Signed-off-by: Ales Novak <alnovak@suse.com>
Signed-off-by: Ales Novak <alnovak@suse.com>
Signed-off-by: Ales Novak <alnovak@suse.com>
Signed-off-by: Ales Novak <alnovak@suse.com>
Signed-off-by: Ales Novak <alnovak@suse.com>
Signed-off-by: Ales Novak <alnovak@suse.com>
Signed-off-by: Ales Novak <alnovak@suse.com>
Signed-off-by: Ales Novak <alnovak@suse.com>
- optimize if-else and returns
- removal of stray passes
- pythonic way of creating dict
- removal of unecessary argument Port.setup#root
- fix shadowing by UIPassthruNode.ui_command_clear_ids#clear

Signed-off-by: Ales Novak <alnovak@suse.com>
- there's no way around getting configshell _exit value
- we're not going to reorganize nvmet.nvme to split it into several
  modules at this point

Signed-off-by: Ales Novak <alnovak@suse.com>
Signed-off-by: Ales Novak <alnovak@suse.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant