Merge pull request #401 from zed-industries/fail-ci-on-warnings

Nathan Sobo created 4 years ago

Cause tests to fail on CI when there are warnings

Change summary

.github/workflows/ci.yml | 2 ++
crates/project/src/fs.rs | 3 ++-
2 files changed, 4 insertions(+), 1 deletion(-)

Detailed changes

.github/workflows/ci.yml

@@ -19,6 +19,8 @@ jobs:
   tests:
     name: Run tests
     runs-on: self-hosted
+    env:
+      RUSTFLAGS: -D warnings
     steps:
       - name: Install Rust
         uses: actions-rs/toolchain@v1

crates/project/src/fs.rs

@@ -1,7 +1,6 @@
 use anyhow::{anyhow, Result};
 use fsevent::EventStream;
 use futures::{Stream, StreamExt};
-use postage::prelude::Sink as _;
 use smol::io::{AsyncReadExt, AsyncWriteExt};
 use std::{
     io,
@@ -164,6 +163,8 @@ impl FakeFsState {
     }
 
     async fn emit_event(&mut self, paths: &[&Path]) {
+        use postage::prelude::Sink as _;
+
         let events = paths
             .iter()
             .map(|path| fsevent::Event {