chore: Remove irrelevant TODO about activity cancellation

bergundy · bergundy · commit 9afd77fa6dec · 2021-07-25T17:32:35.000+03:00
diff --git a/packages/worker/src/activity.ts b/packages/worker/src/activity.ts
@@ -56,7 +56,6 @@ export class Activity {
         if (this.cancelRequested) {
           // Either a CancelledError that we threw or AbortError from AbortController
           if (err instanceof CancelledError || (err.name === 'AbortError' && err.type === 'aborted')) {
-            // TODO: should we only allow CancelledError to count as valid cancellation?
             return { canceled: {} };
           }
         }

Original file line number	Diff line number	Diff line change
`@@ -56,7 +56,6 @@ export class Activity {`
`56`	`56`	`if (this.cancelRequested) {`
`57`	`57`	`// Either a CancelledError that we threw or AbortError from AbortController`
`58`	`58`	`if (err instanceof CancelledError \|\| (err.name === 'AbortError' && err.type === 'aborted')) {`
`59`		`- // TODO: should we only allow CancelledError to count as valid cancellation?`
`60`	`59`	`return { canceled: {} };`
`61`	`60`	`}`
`62`	`61`	`}`