Skip to content

Commit

Permalink
fix: fix Concatenation test by accepting new insta snapshot
Browse files Browse the repository at this point in the history
The new behavior is reasonable. In case of a double empty concatenation,
which is already pretty uncommon, the lint would suggest to remove the
second empty concat first instead of the first empty concat.
  • Loading branch information
RobWalt committed Jan 10, 2025
1 parent 8b2759a commit 5f5a639
Showing 1 changed file with 4 additions and 6 deletions.
10 changes: 4 additions & 6 deletions bin/tests/snapshots/main__empty_list_concat.snap
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
---
source: bin/tests/main.rs
expression: "&out"

expression: "& out"
---
[W23] Warning: Unnecessary concatenation with empty list
╭─[data/empty_list_concat.nix:6:4]
Expand Down Expand Up @@ -32,10 +31,9 @@ expression: "&out"
· ╰──────── Concatenation with the empty list, [], is a no-op
────╯
[W23] Warning: Unnecessary concatenation with empty list
╭─[data/empty_list_concat.nix:15:4]
╭─[data/empty_list_concat.nix:15:10]
15 │ ([] ++ [] ++ [])
· ────┬───
· ╰───── Concatenation with the empty list, [], is a no-op
· ────┬───
· ╰───── Concatenation with the empty list, [], is a no-op
────╯

0 comments on commit 5f5a639

Please sign in to comment.