sindresorhus/p-queue

Use `EventTarget` when targeting Node.js 26

Open

#162 opened on Feb 1, 2022

View on GitHub
 (7 comments) (1 reaction) (0 assignees)TypeScript (4,211 stars) (206 forks)user submission
enhancementhelp wanted

Description

EventTarget is a built-in way to do events in Node.js 16+. This could be used instead of eventemitter3 when Node.js 14 is EOL.

Contributor guide

Tech stack
typescriptnodejs
Domain
backenddeveloper experience
Issue type
refactor
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
Node.js event systemTypeScriptp queue codebase structure
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
65
Research direction
Review the current event handling in the p queue source code, likely in src/index.ts. The issue suggests replacing the eventemitter3 dependency with Node.js built in EventTarget. Check the Node.js 14 EOL status (it reached EOL in April 2023). The change would involve modifying the event emitter implementation, updating tests, and ensuring backward compatibility. Look at the comments for any additional requirements or concerns from maintainers.