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

Remove formatter incompatibility warning for ISC001 #15123

Merged
merged 1 commit into from
Jan 3, 2025

Conversation

MichaReiser
Copy link
Member

Summary

Remove the formatter incompatility warning from ISC001 because the formatter now:

  • Tries to join implicit concatenated strings that fit on a single line
  • Preserves implicit concatenated strings as mulitline for the few cases where the automatic-joining isn't supported

This PR adds a warning for lint.flake8-implicit-str-concat.allow-multiline = false if ISC001 is disabled.
This is not a new incompatibility, it's just that we forgot to add it.

Test Plan

See CLI tests

@MichaReiser MichaReiser force-pushed the micha/remove-isc001-warn branch from f9e0ebd to e777a2f Compare December 23, 2024 12:34
@MichaReiser MichaReiser added the documentation Improvements or additions to documentation label Dec 23, 2024
Copy link
Contributor

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

@MichaReiser MichaReiser marked this pull request as ready for review December 23, 2024 12:46
Base automatically changed from micha/ruff-2025 to ruff-0.9 January 3, 2025 13:16
@MichaReiser MichaReiser force-pushed the micha/remove-isc001-warn branch from e777a2f to aebcf27 Compare January 3, 2025 13:18
@MichaReiser MichaReiser merged commit f2f0ef7 into ruff-0.9 Jan 3, 2025
20 of 21 checks passed
@MichaReiser MichaReiser deleted the micha/remove-isc001-warn branch January 3, 2025 13:24
@MichaReiser MichaReiser mentioned this pull request Jan 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants