sindresorhus/aggregate-error

Add message argument

Open

#12 ouverte le 10 mars 2020

Voir sur GitHub
 (1 commentaire) (1 réaction) (0 assignés)JavaScript (247 stars) (18 forks)user submission
enhancementhelp wanted

Description

Add a message argument to bring the constructor in line with the TC39 AggregateError proposal.

Before

new AggregateError(errors)

After

new AggregateError(errors[, message])

The message is simply written to error.message property.

Guide contributeur

Stack technique
javascript
Domaine
api
Type d'issue
feature
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Basic JavaScriptFamiliarity with Error classes
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
90
Direction de recherche
The issue proposes adding an optional 'message' parameter to the AggregateError constructor, aligning with the TC39 proposal. The change is to modify the constructor to accept a second argument, `message`, and assign it to `this.message`. First, examine the current constructor signature in the repository's source file (likely index.js). Then, update the constructor to handle the new parameter, ensuring backward compatibility when only errors are provided. Finally, add a test case for the new functionality.