Skip to content

Avoid no-self-use for attrs-style validators#13166

Merged
charliermarsh merged 1 commit intomainfrom
charlie/conflict
Aug 30, 2024
Merged

Avoid no-self-use for attrs-style validators#13166
charliermarsh merged 1 commit intomainfrom
charlie/conflict

Conversation

@charliermarsh
Copy link
Copy Markdown
Member

Summary

Closes #12568.

@charliermarsh charliermarsh added the rule Implementing or modifying a lint rule label Aug 30, 2024
@charliermarsh charliermarsh merged commit a73bebc into main Aug 30, 2024
@charliermarsh charliermarsh deleted the charlie/conflict branch August 30, 2024 16:39
@github-actions
Copy link
Copy Markdown
Contributor

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

ℹ️ ecosystem check detected linter changes. (+0 -1 violations, +0 -0 fixes in 1 projects; 53 projects unchanged)

apache/airflow (+0 -1 violations, +0 -0 fixes)

ruff check --no-cache --exit-zero --ignore RUF9 --output-format concise --preview --select ALL

- airflow/decorators/task_group.py:71:9: PLR6301 Method `_validate` could be a function, class method, or static method

Changes by rule (1 rules affected)

code total + violation - violation + fix - fix
PLR6301 1 0 1 0 0

@charliermarsh
Copy link
Copy Markdown
Member Author

Ecosystem change is correct!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rule Implementing or modifying a lint rule

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PLR6301 not aware of attrs validation decorator

1 participant