sindresorhus/aggregate-error

Add message argument

Open

#12 aberto em 10 de mar. de 2020

Ver no GitHub
 (1 comment) (1 reaction) (0 assignees)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.

Guia do colaborador

Pilha de tecnologia
javascript
Domain
api
Tipo Issue
feature
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 hours
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
Basic JavaScriptFamiliarity with Error classes
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
90
Direção de pesquisa
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.