mholt/PapaParse

remove jquery stuff

Open

#841 ouverte le 14 nov. 2020

Voir sur GitHub
 (0 commentaires) (8 réactions) (0 assignés)JavaScript (1 157 forks)batch import
enhancementgood first issuehelp wanted

Métriques du dépôt

Stars
 (11 877 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

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

Guide contributeur

Direction de recherche
Examinez le bloc de code jQuery aux lignes 87-179 de papaparse.js pour comprendre son rôle (probablement un ancien wrapper de plugin jQuery). Vérifiez qu'aucune autre partie de la base de code ne dépend de ce code, par exemple en vérifiant les imports ou les fichiers de test. Supprimez complètement le bloc et exécutez la suite de tests existante pour vous assurer que rien ne casse. Comme le problème est inactif, ouvrez une PR avec une description claire de la suppression et mettez à jour toute documentation pertinente.
Stack technique
javascript
Domaine
data
Type d'issue
Refactorisation
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 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80