sindresorhus/del

Async throw error delay

Open

#98 opened on 2019年7月12日

GitHub で見る
 (3 comments) (0 reactions) (0 assignees)JavaScript (1,305 stars) (71 forks)batch import
enhancementhelp wanted

説明

Would you accept a PR that would change how async errors are thrown so the error is thrown after all pending files being deleted are removed? This would help deal with side effects a lot cleaner, especially when testing.

Similar to https://github.com/sindresorhus/del/pull/90.

コントリビューターガイド

技術スタック
javascriptnodejs
領域
tooling
Issue 種別
feature
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3 hours
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
mostly clear
前提条件
Node.js async/awaiterror handling patterns
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
60
調査方針
Investigate the existing error handling in the del library, focusing on the async file deletion logic. Review linked PR #90 for a similar approach. Test how errors are currently thrown during pending deletions and propose a change to defer the error until all deletions complete.