Skip to content

Remove textwrap.dedent - #685

Merged
mristin merged 1 commit into
mainfrom
remove-textwrap-dedent
Sep 10, 2026
Merged

mristin merged 1 commit into
mainfrom
remove-textwrap-dedent

Conversation

@mristin

@mristin mristin commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

textwrap.dedent wraps source literals in an extra call and forces the content to be indented to match the surrounding code, which makes it harder to read and edit. Instead, we write triple-quoted strings flush-left at column 0 instead, which are easier to maintain and copy/paste.

``textwrap.dedent`` wraps source literals in an extra call and forces
the content to be indented to match the surrounding code, which makes it
harder to read and edit. Instead, we write triple-quoted strings
flush-left at column 0 instead, which are easier to maintain and
copy/paste.
@mristin
mristin merged commit 24a77fe into main Sep 10, 2026
5 checks passed
@mristin
mristin deleted the remove-textwrap-dedent branch September 10, 2026 15:09
@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 34493015465

Warning

Build has drifted: This PR's base is out of sync with its target branch, so coverage data may include unrelated changes.
Quick fix: rebase this PR. Learn more →

Coverage remained the same at 85.242%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: 3 uncovered changes across 1 file (1 of 4 lines covered, 25.0%).
  • No coverage regressions found.

Uncovered Changes

File Changed Covered %
aas_core_codegen/csharp/lib/_generate_verification.py 4 1 25.0%

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 39816
Covered Lines: 33940
Line Coverage: 85.24%
Coverage Strength: 2.56 hits per line

💛 - Coveralls

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants