Specify the amount of spaces before inline comments #186
+526
−35
Conversation
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
* feat: add whitespace option for string concatenation * Fix concat operator formatting and add context checks Address Qodo review comments: - Add level and end_of_delim checks to avoid formatting // inside strings/comments - Remove unsafe trailing rstrip that could strip string literal whitespace - Enable concat spacing at whitespace level 4 - Add comprehensive test coverage for concat operator formatting * Restore README.md from master * fix(indent): realign mis-indented nested blocks * Update example.f90 * Update example.f90 * Fix indentation in nested DO loops * Update expected_results * Update expected_results
* fix(indent): keep indent when at line limit * Fix padding calculation for line length overflow Address Qodo review: recompute padding in elif block to correctly right-align lines at the line length limit when indent would overflow. Update test expectations to match corrected behavior. * Improve line splitting for long logical lines * Guard line splitting for syntax-safe breakpoints * Improve auto line splitting for strings and continuation layout * Update expected results for nested-loop indentation * Ensure indent pass splits long lines * Test indent-only line splitting * Refactor auto split insertion * Split inline comments before enforcing line limits
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
To enhance the great CLI option
--strip-commentsfurther the additional option--comment-spacing=Nwas implemented. That way people could e.g. specify--comment-spacing=2to have a more 'pythonic' style for comments.