mholt/PapaParse

remove jquery stuff

Open

#841 aperta il 14 nov 2020

Vedi su GitHub
 (0 commenti) (8 reazioni) (0 assegnatari)JavaScript (1157 fork)batch import
enhancementgood first issuehelp wanted

Metriche repository

Star
 (11.877 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

I think this tool should be as vanilla as possible and only do one thing web workers, node and deno don't need this unnecessary stuff

that means removing this jQuery bit https://github.com/mholt/PapaParse/blob/5747da6c9927642743d6d44d49689be2d2fa0b10/papaparse.js#L87-L179

most ppl have moved away from the good old days of jquery now where MVC have taken over such as vue, angular, react, sapper etc

Guida contributor

Direzione di ricerca
Esamina il blocco di codice jQuery alle righe 87-179 di papaparse.js per comprenderne il ruolo (probabilmente un vecchio wrapper di plugin jQuery). Verifica che nessun'altra parte della codebase dipenda da questo codice, ad esempio controllando le importazioni o i file di test. Rimuovi completamente il blocco ed esegui la suite di test esistente per assicurarti che nulla si rompa. Poiché il problema è inattivo, apri una PR con una descrizione chiara della rimozione e aggiorna eventuali documentazioni pertinenti.
Tech stack
javascript
Dominio
data
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80