Add support for rm_file in GitHubFileSystem implementation

fsspec/implementations/github.py

@@ -265,3 +265,33 @@ def _open(
             cache_options=cache_options,
             **kwargs,
         )
+
+    def _rm(self, path):
+        if not self.username:
+            raise ValueError("Authentication required")
+
+        path = self._strip_protocol(path).lstrip("/")
+
+        # Get the file SHA
+        url = self.content_url.format(
+            org=self.org, repo=self.repo, path=path, sha=self.root
+        )
+        r = requests.get(url, timeout=self.timeout, **self.kw)
+        if r.status_code == 404:
+            raise FileNotFoundError(path)
+        r.raise_for_status()
+        content_json = r.json()
+        sha = content_json["sha"]
+
+        # Delete the file using GitHub API
+        delete_url = url
+        data = {
+            "message": f"Delete {path}",
+            "sha": sha,
+            "branch": self.root,
+        }
+        r = requests.delete(delete_url, json=data, timeout=self.timeout, **self.kw)
+        if r.status_code not in (200, 204):
+            r.raise_for_status()
+
+        self.invalidate_cache(path)

fsspec/implementations/tests/test_github.py

@@ -1,4 +1,5 @@
 import fsspec
+import pytest
 
 
 def test_github_open_small_file():
@@ -46,3 +47,17 @@ def test_github_ls():
     expected = {"brain_networks.csv", "mpg.csv", "penguins.csv", "README.md", "raw"}
     # check if the result is a subset of the expected files
     assert expected.issubset(ls_result)
+
+
+def test_github_rm():
+    # trying to remove a file without passing authentication should raise ValueError
+    fs = fsspec.filesystem("github", org="mwaskom", repo="seaborn-data")
+    with pytest.raises(ValueError):
+        fs.rm("mpg.csv")
+
+    # trying to remove a file which doesn't exist should raise FineNotFoundError
+    fs = fsspec.filesystem(
+        "github", org="mwaskom", repo="seaborn-data", username="user", token="token"
+    )
+    with pytest.raises(FileNotFoundError):
+        fs.rm("/this-file-doesnt-exist")