Skip to content

Apply pylint suggestions to base.py#175

Closed
furtib wants to merge 1 commit intoEricsson:mainfrom
furtib:fix-lint-base
Closed

Apply pylint suggestions to base.py#175
furtib wants to merge 1 commit intoEricsson:mainfrom
furtib:fix-lint-base

Conversation

@furtib
Copy link
Contributor

@furtib furtib commented Jan 30, 2026

Why:
We want to add a passing pylint workflow in CI.

What:

  • Applied most pylint suggestion, to test/common/base.py
  • Fixed error with logging
  • Exceptions:
    • consider-using-with on CodeChecker server related functions. (They are by design separated)

Addresses:
#153

@furtib furtib requested a review from Szelethus January 30, 2026 10:36
@furtib furtib self-assigned this Jan 30, 2026
@furtib furtib added non-functional change ☮️ The patch doesn't change any functionality, e.g. refactoring, documentation, test-only. test ☑️ Adding or refactoring tests labels Jan 30, 2026
@furtib
Copy link
Contributor Author

furtib commented Feb 2, 2026

Combined with #177

@furtib furtib closed this Feb 2, 2026
@furtib furtib deleted the fix-lint-base branch February 2, 2026 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

non-functional change ☮️ The patch doesn't change any functionality, e.g. refactoring, documentation, test-only. test ☑️ Adding or refactoring tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments