Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add a few more checks to the pylint ignore list #208

Merged
merged 1 commit into from
Jan 4, 2024
Merged

Add a few more checks to the pylint ignore list #208

merged 1 commit into from
Jan 4, 2024

Conversation

llucax
Copy link
Contributor

@llucax llucax commented Dec 29, 2023 •
edited
Loading

This is to avoid double reporting of some checks that are already performed by flake8.

Also sort the list alphabetically so it is easier to find a specific check.

@llucax llucax requested a review from a team as a code owner December 29, 2023 10:25
@llucax llucax requested a review from shsms December 29, 2023 10:25
@llucax llucax self-assigned this Dec 29, 2023
@llucax llucax added this to the v0.9.0 milestone Dec 29, 2023
@github-actions github-actions bot added the part:tests Affects the unit, integration and performance (benchmarks) tests label Dec 29, 2023
@llucax llucax added part:tooling Affects the development tooling (CI, deployment, dependency management, etc.) part:cookiecutter Affects the generation of projects using cookiecutter part:ci Affects the GitHub workflow and other parts for running CI labels Dec 29, 2023
@llucax llucax enabled auto-merge December 29, 2023 10:26
@github-actions github-actions bot removed part:tooling Affects the development tooling (CI, deployment, dependency management, etc.) part:cookiecutter Affects the generation of projects using cookiecutter part:ci Affects the GitHub workflow and other parts for running CI labels Dec 29, 2023
auto-merge was automatically disabled January 2, 2024 12:21

Merge queue setting changed

@llucax
Add a few more checks to the pylint ignore list
e4b9301 
This is to avoid double reporting of some checks that are already
performed by `flake8`.

Also sort the list alphabetically so it is easier to find a specific
check.

Signed-off-by: Leandro Lucarella <[email protected]>
@llucax llucax added this pull request to the merge queue Jan 4, 2024
Merged via the queue into frequenz-floss:v0.x.x with commit e1c7147 Jan 4, 2024
14 checks passed
@llucax llucax deleted the pylint-disable branch January 4, 2024 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@daniel-zullo-frequenz daniel-zullo-frequenz daniel-zullo-frequenz approved these changes

@shsms shsms Awaiting requested review from shsms shsms is a code owner automatically assigned from frequenz-floss/python-sdk-team

Assignees

@llucax llucax

Labels
part:tests Affects the unit, integration and performance (benchmarks) tests
Projects
None yet
Milestone
v0.9.0
Development

Successfully merging this pull request may close these issues.
2 participants
@llucax @daniel-zullo-frequenz