🩹 Polyfill throwIfAborted (#294)

huggingface · Nov 15, 2023 · d5184f7 · d5184f7
1 parent 0c568c7
commit d5184f7
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/.github/workflows/lint-and-test.yml b/.github/workflows/lint-and-test.yml
@@ -75,7 +75,10 @@ jobs:
         working-directory: e2e/ts
         run: |
           npm i -g yarn --force
+          # yarn now looks at root package.json for the package manager...
+          mv ../../package.json ../../package.json.bk
           yarn install --registry http://localhost:4874/
+          mv ../../package.json.bk ../../package.json
 
       - name: E2E test - typescript node project
         working-directory: e2e/ts

diff --git a/packages/hub/src/lib/commit.ts b/packages/hub/src/lib/commit.ts
@@ -129,6 +129,15 @@ export async function* commitIter(params: CommitParams): AsyncGenerator<CommitPr
 	const abortController = new AbortController();
 	const abortSignal = abortController.signal;
 
+	// Polyfill see https://discuss.huggingface.co/t/why-cant-i-upload-a-parquet-file-to-my-dataset-error-o-throwifaborted-is-not-a-function/62245
+	if (!abortSignal.throwIfAborted) {
+		abortSignal.throwIfAborted = () => {
+			if (abortSignal.aborted) {
+				throw new DOMException("Aborted", "AbortError");
+			}
+		};
+	}
+
 	if (params.abortSignal) {
 		params.abortSignal.addEventListener("abort", () => abortController.abort());
 	}