Skip to content

lint: remove empty line - #1232

Merged
lgfa29 merged 1 commit into
masterfrom
luiz/fix-clippy-empty-line
Oct 22, 2026
Merged

lgfa29 merged 1 commit into
masterfrom
luiz/fix-clippy-empty-line

Conversation

@lgfa29

@lgfa29 lgfa29 commented Oct 22, 2026

Copy link
Copy Markdown
Member

clippy is not happy about this empty line.

warning: empty line after outer attribute
  --> lib/propolis/src/hw/virtio/mod.rs:40:1
   |
40 | / #[cfg(target_os = "illumos")]
41 | |
   | |_^
42 |   mod os {
   |   ------ the attribute applies to this module
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/rust-1.97.0/index.html#empty_line_after_outer_attr
   = note: `#[warn(clippy::empty_line_after_outer_attr)]` on by default
   = help: if the empty line is unintentional, remove it

clippy is not happy about this empty line.

```
warning: empty line after outer attribute
  --> lib/propolis/src/hw/virtio/mod.rs:40:1
   |
40 | / #[cfg(target_os = "illumos")]
41 | |
   | |_^
42 |   mod os {
   |   ------ the attribute applies to this module
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/rust-1.97.0/index.html#empty_line_after_outer_attr
   = note: `#[warn(clippy::empty_line_after_outer_attr)]` on by default
   = help: if the empty line is unintentional, remove it
```
@lgfa29
lgfa29 requested review from iximeow and rzezeski October 22, 2026 21:55
@lgfa29
lgfa29 merged commit 79ae305 into master Oct 22, 2026
14 checks passed
@lgfa29
lgfa29 deleted the luiz/fix-clippy-empty-line branch October 22, 2026 23:22
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