iamkun/dayjs

🎁 2.0 Milestones【Breaking Change】

Aperta

#1281 aperta il 19 dic 2020

Vedi su GitHubGuida contributor
 (38 commenti) (17 reazioni) (0 assegnatari)JavaScript (2463 fork)batch import
Breaking Changediscussionhelp wanted🚨 High Priority

Metriche repository

Star
 (48.666 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

We want to make some changes to make Day.js modern and in a standard JS way.

And this would be a breaking change because of these changes below:

Make TypeScript definition the same behavior as JS code

Update types/index.d.ts to export default dayjs;

Mark the correct ES6 Module entry

Update package.json "module" to "esm/index", (won't introduce in "dayjs", see https://github.com/iamkun/dayjs/issues/1281#issuecomment-754381268)

Publish a new NPM package, maybe "dayjs-esm" to ship an ESM module with package.json module entry pointed to. ref https://github.com/iamkun/dayjs/issues/1242#issuecomment-734321931

Guida contributor

Direzione di ricerca
Indaga sulle due modifiche proposte: aggiornare types/index.d.ts per esportare dayjs come default e creare un pacchetto ESM separato (dayjs esm). Esamina le discussioni correlate negli issue #1242 e #1281 commenti, e analizza l'attuale package.json e le definizioni dei tipi. Considera l'impatto sugli utenti esistenti e come mantenere la compatibilità all'indietro dove possibile.
Tech stack
javascripttypescript
Dominio
toolingrelease
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
TypeScriptnpm
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30