Skip to content

Disable the new check-class-attributes check in pydoclint 0.5.3#288

Merged
llucax merged 1 commit intofrequenz-floss:v0.x.xfrom
llucax:pydoclint-update
Jul 3, 2024
Merged

Disable the new check-class-attributes check in pydoclint 0.5.3#288
llucax merged 1 commit intofrequenz-floss:v0.x.xfrom
llucax:pydoclint-update

Conversation

@llucax
Copy link
Copy Markdown
Contributor

@llucax llucax commented Jul 3, 2024

We use a different way to document class attributes, so this check is not useful for us.

@llucax llucax requested a review from a team as a code owner July 3, 2024 07:43
@llucax llucax requested a review from shsms July 3, 2024 07:43
@github-actions github-actions Bot added the part:template Affects the cookiecutter template files label Jul 3, 2024
@llucax llucax self-assigned this Jul 3, 2024
@llucax llucax added this to the v0.10.0 milestone Jul 3, 2024
@llucax llucax added type:bug Something isn't working part:tooling Affects the development tooling (CI, deployment, dependency management, etc.) labels Jul 3, 2024
@llucax llucax enabled auto-merge July 3, 2024 07:45
@llucax llucax force-pushed the pydoclint-update branch from 6ed357e to a025fa2 Compare July 3, 2024 07:56
shsms
shsms previously approved these changes Jul 3, 2024
We use a different way to document class attributes, so this check is
not useful for us.

Signed-off-by: Leandro Lucarella <luca-frequenz@llucax.com>
@llucax
Copy link
Copy Markdown
Contributor Author

llucax commented Jul 3, 2024

Rebased, needs a new approval.

@llucax llucax requested a review from shsms July 3, 2024 11:14
@llucax llucax added this pull request to the merge queue Jul 3, 2024
Merged via the queue into frequenz-floss:v0.x.x with commit 2ac253a Jul 3, 2024
@llucax llucax deleted the pydoclint-update branch July 3, 2024 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

part:template Affects the cookiecutter template files part:tooling Affects the development tooling (CI, deployment, dependency management, etc.) type:bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants