Skip to content

Commit

Permalink
TODO suppression must be on the same line as todo comment
Browse files Browse the repository at this point in the history
  • Loading branch information
ladatz committed Sep 27, 2023
1 parent fb759ba commit 21596bc
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions ess/src/ess.rs
Original file line number Diff line number Diff line change
Expand Up @@ -369,8 +369,7 @@ where
ClientId: Eq + Hash,
EventId: Eq + Hash,
{
// DevSkim: ignore DS176209
// TODO address too many arguments
// DevSkim: ignore DS176209 TODO address too many arguments
#[allow(clippy::too_many_arguments)]
async fn serve_with_handlers(
mut self,
Expand Down Expand Up @@ -597,8 +596,7 @@ mod tests {
// act
sut.publish(&EVENT_ID, Event(EVENT_ID, SeqNum(0), DATA1));
sut.publish(&EVENT_ID, Event(EVENT_ID, SeqNum(0), DATA2));
// DevSkim: ignore DS176209
// TODO investigate how to avoid sleeping here
// DevSkim: ignore DS176209 TODO investigate how to avoid sleeping here
// see also: https://github.com/tokio-rs/tokio/issues/2443
std::thread::sleep(Duration::from_secs_f64(0.1));
// assert
Expand Down

0 comments on commit 21596bc

Please sign in to comment.