Skip to content

Commit

Permalink
Switch off warnings for legacy client build
Browse files Browse the repository at this point in the history
As the sync clients is not developed anymore, and once in a while there are new clippy warnings added,
this disables all of them so we don't have to change sync client codebase after CI toolchain upgrade.
  • Loading branch information
Jarema authored Jul 14, 2023
1 parent 6f173a3 commit 014c700
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
2 changes: 1 addition & 1 deletion async-nats/src/service/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -694,7 +694,7 @@ impl Request {
let elapsed = self.issued.elapsed();
let mut stats = self.stats.lock().unwrap();
// let mut stats = stats.endpoints.entry(key)
let mut stats = stats.endpoints.get_mut(self.endpoint.as_str()).unwrap();
let stats = stats.endpoints.get_mut(self.endpoint.as_str()).unwrap();
stats.requests += 1;
stats.processing_time += elapsed;
stats.average_processing_time = stats.processing_time.checked_div(2).unwrap();
Expand Down
5 changes: 3 additions & 2 deletions nats/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -114,7 +114,6 @@
//! ```

#![cfg_attr(docsrs, feature(doc_cfg))]
#![cfg_attr(test, deny(warnings))]
#![cfg_attr(
feature = "fault_injection",
deny(
Expand Down Expand Up @@ -188,7 +187,9 @@
clippy::wildcard_enum_match_arm,
clippy::module_name_repetitions
)]

// As this is a deprecated client, we don't want warnings from new lints to make CI red.
#![allow(clippy::all)]
#![allow(warnings)]
/// Async-enabled NATS client.
pub mod asynk;

Expand Down

0 comments on commit 014c700

Please sign in to comment.