fix: remove empty line after doc comment in import_rewriter#284
Merged
GTC6244 merged 1 commit intofeature/cpl-216-unit-tests-in-lit-api-serverfrom Apr 14, 2026
Merged
Conversation
The empty line between the doc comment and `rewrite_imports` function triggered `clippy::empty_line_after_doc_comments`, failing CI. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR fixes a CI-clippy failure by removing an empty line between a doc comment block and the rewrite_imports function in the Lit Actions import rewriter module.
Changes:
- Removes the blank line after the
rewrite_importsdoc comment to satisfyclippy::empty_line_after_doc_comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
GTC6244
changed the base branch from
next
to
feature/cpl-216-unit-tests-in-lit-api-server
GTC6244
merged commit 151c292
into
feature/cpl-216-unit-tests-in-lit-api-server
14 checks passed
GTC6244
added a commit
that referenced
this pull request
Apr 14, 2026
* test: add unit tests across lit-api-server modules (CPL-216) Add 100 unit tests covering parsing utilities, cryptographic models, AES encryption, API status helpers, chain info lookups, billing helpers, and derivation path generation. Tests exercise round-trip conversions, edge cases, error paths, and enum exhaustiveness. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * cargo fmt * fix: remove empty line after doc comment to fix clippy (#284) The empty line between the doc comment and `rewrite_imports` function triggered `clippy::empty_line_after_doc_comments`, failing CI. Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
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
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.
Summary
rewrite_importsfunction inlit-actions/server/import_rewriter.rsclippy::empty_line_after_doc_commentslint error that was failing CI on PR test: add unit tests across lit-api-server modules (CPL-216) #277Test plan
🤖 Generated with Claude Code