Skip to content
New issue

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

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

Already on GitHub? # to your account

[flake8-pytest-style] Add a space after comma in CSV output (PT006) #12853

Merged
merged 1 commit into from
Aug 13, 2024

Conversation

uranusjr
Copy link
Contributor

Summary

See #12703. This only addresses the first bullet point, adding a space after the comma in the suggested fix from list/tuple to string.

Test Plan

Updated the snapshots and compared.

Copy link
Contributor

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

Copy link
Member

@dhruvmanila dhruvmanila left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems reasonable, thanks!

@dhruvmanila dhruvmanila added the fixes Related to suggested fixes for violations label Aug 13, 2024
@dhruvmanila dhruvmanila changed the title Add a space after comma in CSV output [flake8-pytest-style] Add a space after comma in CSV output (PT006) Aug 13, 2024
@dhruvmanila dhruvmanila merged commit 82a3e69 into astral-sh:main Aug 13, 2024
20 checks passed
@uranusjr uranusjr deleted the flake8-pytest-csv-comma-space branch August 13, 2024 08:06
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
fixes Related to suggested fixes for violations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants