caolan/async

async package should distribute both cjs and esm

Open

#1.879 geöffnet am 15. Nov. 2022

Auf GitHub ansehen
 (1 Kommentar) (7 Reaktionen) (0 zugewiesene Personen)JavaScript (2.398 Forks)batch import
help wanted

Repository-Metriken

Stars
 (28.067 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

There is no real need for async-es package. It ends up complicating async usage downstream especially for projects which emit both cjs and esm. And to complicate things, async-es has a main entry which points to esm (without indicating type: 'module'), so it can't be used for cjs scenarios.

Recommended:

Update the async package.json with:

  1. main pointing to commonjs. (this is only for backwards compatibility.)
  2. module pointing to esm. (this is only for backwards compatibility.)
  3. exports map explicitly listing the valid imports and the cjs/esm/d.ts files to resolve to.

Contributor Guide

Research-Richtung
Prüfe die aktuelle package.json und verstehe, wie man duale CJS/ESM Exporte einrichtet. Siehe die Node.js Dokumentation zu package.json Exporten. Implementiere dann die Änderungen wie im Problem beschrieben.
Tech Stack
javascript
Domain
full stack
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitNode.js
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75