Run CI on PRs (#336)

* .

* clippy fixes

* no need for a path filter

Author: AdamGS

.github/workflows/object_store.yml renamed to .github/workflows/ci.yml

@@ -16,8 +16,8 @@
 # under the License.
 
 ---
-# tests for `object_store` crate
-name: object_store
+
+name: CI
 
 concurrency:
   group: ${{ github.repository }}-${{ github.head_ref || github.sha }}-${{ github.workflow }}
@@ -29,9 +29,6 @@ on:
     branches:
       - main
   pull_request:
-    paths:
-      - object_store/**
-      - .github/**
 
 jobs:
   clippy:
@@ -77,7 +74,7 @@ jobs:
     steps:
       - uses: actions/checkout@v4
       - name: Run cargo doc
-        run: cargo doc --document-private-items --no-deps --workspace --all-features
+        run: cargo doc --document-private-items --no-deps --all-features
 
   # test the crate
   # This runs outside a container to workaround lack of support for passing arguments

src/delimited.rs

@@ -109,7 +109,7 @@ impl LineDelimiter {
                 }
             },
         };
-        let end_offset = record_ends.last().unwrap_or(start_offset);
+        let end_offset = record_ends.next_back().unwrap_or(start_offset);
         if start_offset != end_offset {
             self.complete.push_back(val.slice(start_offset..end_offset));
         }

src/integration.rs

@@ -657,7 +657,7 @@ pub async fn put_opts(storage: &dyn ObjectStore, supports_update: bool) {
             .put_opts(&path, "c".into(), PutMode::Update(v1.clone().into()).into())
             .await
             .unwrap_err();
-        assert!(matches!(err, Error::NotImplemented { .. }), "{err}");
+        assert!(matches!(err, Error::NotImplemented), "{err}");
 
         return;
     }