Skip to content

Commit

Permalink
Merge pull request #92 from svix/jplatte/warnings
Browse files Browse the repository at this point in the history
Fix future incompatibility warnings regarding never type fallback
  • Loading branch information
svix-jplatte authored Aug 6, 2024
2 parents e30bfee + ad59251 commit 9f595ad
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 4 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ jobs:
run: cargo hack --each-feature clippy --all-targets -- -D warnings

- name: Start test dependencies
run: docker-compose -f "./testing-docker-compose.yml" up -d
run: docker compose -f "./testing-docker-compose.yml" up -d

- name: Run tests
env:
Expand All @@ -55,7 +55,7 @@ jobs:
run: cargo test --all-features

- name: Stop test dependencies
run: docker-compose -f "./testing-docker-compose.yml" down
run: docker compose -f "./testing-docker-compose.yml" down

typos:
name: Check for typos
Expand Down
3 changes: 2 additions & 1 deletion omniqueue/src/backends/redis/fallback.rs
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,8 @@ impl<R: RedisConnection> Acker for RedisFallbackAcker<R> {

self.already_acked_or_nacked = true;

self.redis
let _: () = self
.redis
.get()
.await
.map_err(QueueError::generic)?
Expand Down
3 changes: 2 additions & 1 deletion omniqueue/src/backends/redis/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -550,7 +550,8 @@ impl<R: RedisConnection> RedisProducer<R> {
pub async fn send_raw_scheduled(&self, payload: &[u8], delay: Duration) -> Result<()> {
let timestamp = unix_timestamp(SystemTime::now() + delay).map_err(QueueError::generic)?;

self.redis
let _: () = self
.redis
.get()
.await
.map_err(QueueError::generic)?
Expand Down

0 comments on commit 9f595ad

Please sign in to comment.