FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[`pycodestyle`] Make example not raise unnecessary `SyntaxError` (`E114`) by MeGaGiGaGon · Pull Request #19190 · astral-sh/ruff · GitHub

Repository navigation

[pycodestyle] Make example not raise unnecessary SyntaxError (E114) - #19190

Merged
ntBre merged 4 commits into
astral-sh:mainfrom
MeGaGiGaGon:patch-1
Jul 8, 2025
Merged

ntBre merged 4 commits into
astral-sh:mainfrom
MeGaGiGaGon:patch-1

Conversation

Copy link
Copy Markdown
Contributor

Summary

Part of #18972

This PR makes indentation-with-invalid-multiple-comment (E114)'s example not raise a syntax error by adding a 4 space indented .... The example still gave E114 without this, but adding the ... both makes the change in indentation of the comment clearer, and makes it not give a SyntaxError.

Test Plan

N/A, no functionality/tests affected

MeGaGiGaGon changed the title [pycodestyle] Make example error out-of-the-box (E114) [pycodestyle] Make example not raise unnecessary SyntaxError (E114) Jul 7, 2025

github-actions Bot commented Jul 7, 2025

Copy link
Copy Markdown
Contributor

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

Formatter (stable)

✅ ecosystem check detected no format changes.

Formatter (preview)

✅ ecosystem check detected no format changes.

ntBre self-requested a review July 8, 2025 02:32
ntBre added the documentation Improvements or additions to public-facing documentation label Jul 8, 2025
ntBre merged commit d78d10d into astral-sh:main Jul 8, 2025
MeGaGiGaGon deleted the patch-1 branch July 8, 2025 14:34
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to public-facing documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL