Skip to content

Commit aa8a0ad

Browse files
committed
Allow clippy::unwrap-or-default because its usually wrong
`or_default` is generally less readable than writing out the thing we're writing, as `Default` is opaque but explicit constructors generally are not. Thus, we ignore the clippy lint (ideally we could invert it and ban the use of `Default` in the crate entirely but alas).
1 parent cd4f665 commit aa8a0ad

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

.github/workflows/build.yml

+2
Original file line numberDiff line numberDiff line change
@@ -247,6 +247,8 @@ jobs:
247247
- name: Run default clippy linting
248248
run: |
249249
RUSTFLAGS='-D warnings' cargo clippy -- \
250+
`# Things that are more readable in the non-clippy-enforce version` \
251+
-A clippy::unwrap-or-default
250252
`# Errors` \
251253
-A clippy::erasing_op \
252254
-A clippy::never_loop \

0 commit comments

Comments
 (0)