refactor: remove fetch! macro because useless and hard to read

LiterateInk · Dec 31, 2024 · 4f8c26c · 4f8c26c
1 parent adae058
commit 4f8c26c
Show file tree

Hide file tree

Showing 8 changed files with 4 additions and 187 deletions.
diff --git a/.github/workflows/publish.yml b/.github/workflows/publish.yml
@@ -8,7 +8,7 @@ on:
 
 jobs:
   publish-js:
-    runs-on: 'ubuntu-latest'
+    runs-on: ubuntu-latest
 
     permissions:
       contents: read
@@ -40,7 +40,7 @@ jobs:
           npm publish --provenance --access public
 
   publish-rust:
-    runs-on: 'ubuntu-latest'
+    runs-on: ubuntu-latest
 
     permissions:
       contents: write

diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -1,6 +1,6 @@
 [workspace]
 resolver = "2"
-members = ["wasm", "fetcher", "fetcher-macros"]
+members = ["wasm", "fetcher"]
 
 [workspace.package]
 version = "0.1.2"

diff --git a/fetcher-macros/Cargo.toml b/fetcher-macros/Cargo.toml
diff --git a/fetcher-macros/examples/fetch.rs b/fetcher-macros/examples/fetch.rs
diff --git a/fetcher-macros/src/lib.rs b/fetcher-macros/src/lib.rs
diff --git a/fetcher/Cargo.toml b/fetcher/Cargo.toml
@@ -12,7 +12,6 @@ serde = { version = "1.0", features = ["derive"] }
 serde_bytes = "0.11"
 http = "1.2"
 url = "2.5"
-fetcher-macros = { package = "literateink-fetcher-macros", version = "0.1", path = "../fetcher-macros" }
 
 [target.'cfg(target_arch = "wasm32")'.dependencies]
 js-sys = "0.3"

diff --git a/fetcher/src/index.ts b/fetcher/src/index.ts
@@ -66,7 +66,7 @@ export const defaultFetcher: Fetcher = async (req) => {
   const status = response.status; // should be `u16`
   const bytes = await response.arrayBuffer();
 
-  // Sicne Headers.entries() is not available in all environments
+  // Since `Headers.entries()` is not available in all environments
   // we have to do this dirty hack to get all headers...
   headers = (isHeaders(response.headers)
     // Available in this environment