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.

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

Disable more pylint checks that are also checked by mypy #330

Merged
merged 2 commits into from
Nov 20, 2024

Conversation

llucax
Copy link
Contributor

@llucax llucax commented Nov 20, 2024

pylint has some false positive for these checks, so better to just rely on mypy for them.

@llucax llucax requested a review from a team as a code owner November 20, 2024 10:49
@github-actions github-actions bot added the part:template Affects the cookiecutter template files label Nov 20, 2024
@llucax
Copy link
Contributor Author

llucax commented Nov 20, 2024

@llucax llucax self-assigned this Nov 20, 2024
@llucax llucax added the type:enhancement New feature or enhancement visitble to users label Nov 20, 2024
@llucax llucax added this to the v0.11.0 milestone Nov 20, 2024
@llucax llucax force-pushed the pylint-- branch 2 times, most recently from 1d85c06 to 47f790d Compare November 20, 2024 10:57
@github-actions github-actions bot added the part:nox Affects the configuration of nox label Nov 20, 2024
@llucax llucax requested a review from Marenz November 20, 2024 12:14
@llucax llucax marked this pull request as draft November 20, 2024 12:14
@llucax llucax marked this pull request as ready for review November 20, 2024 13:09
Signed-off-by: Leandro Lucarella <luca-frequenz@llucax.com>
pylint has some false positive for these checks, so better to just rely
on mypy for them.

Signed-off-by: Leandro Lucarella <luca-frequenz@llucax.com>
@llucax llucax added this pull request to the merge queue Nov 20, 2024
Merged via the queue into frequenz-floss:v0.x.x with commit b787200 Nov 20, 2024
14 checks passed
@llucax llucax deleted the pylint-- branch November 20, 2024 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
part:nox Affects the configuration of nox part:template Affects the cookiecutter template files type:enhancement New feature or enhancement visitble to users
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants