Skip to content

Commit

Permalink
Auto merge of #12988 - epage:clippy-type, r=weihanglo
Browse files Browse the repository at this point in the history
chore(ci): Catch naive use of AtomicU64 early

AtomicU64 is not portable and use of it, without `target_has_atomic`, will fail CI for rust-lang/rust when the cargo submodule gets updated (see #12981).

If there is a reason to use it, we could always `allow` in that one case.
  • Loading branch information
bors committed Nov 16, 2023
2 parents 8d7ec64 + 8080b87 commit be9f24d
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions clippy.toml
Original file line number Diff line number Diff line change
Expand Up @@ -6,3 +6,6 @@ disallowed-methods = [
{ path = "std::env::vars", reason = "not recommended to use in Cargo. See rust-lang/cargo#11588" },
{ path = "std::env::vars_os", reason = "not recommended to use in Cargo. See rust-lang/cargo#11588" },
]
disallowed-types = [
{ path = "std::sync::atomic::AtomicU64", reason = "not portable. See rust-lang/cargo#12988" },
]

0 comments on commit be9f24d

Please sign in to comment.