-
-
Notifications
You must be signed in to change notification settings - Fork 747
refactor(formatter): remove useless code in comment printing #16488
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
refactor(formatter): remove useless code in comment printing #16488
Conversation
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
CodSpeed Performance ReportMerging #16488 will not alter performanceComparing Summary
Footnotes
|
036efbe to
289f56e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR refactors the comment printing logic in the formatter by removing unused code. The changes simplify the implementation of multiline comment formatting without affecting functionality.
- Removed unused
source_offsetvariable that was being tracked but never read - Removed unnecessary
format_oncewrapper around a simple for loop - Removed associated lint suppression for
cast_possible_truncationthat was only needed for the removed code
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Merge activity
|
Remove useless code only
289f56e to
fd33a4d
Compare
…ject#16488) Remove useless code only

Remove useless code only