you-dont-need/You-Dont-Need-Lodash-Underscore

Add more suggestions for _.first

Open

#312 ouverte le 2 mai 2021

Voir sur GitHub
 (1 commentaire) (1 réaction) (0 assignés)JavaScript (811 forks)batch import
enhancementhelp wanted

Métriques du dépôt

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

Description

Currently, the suggestion we get for _.first might not be relevant in all cases. Array.prototype.slice() returns an array. But _.first from lodash returns a non-array value. So this line should be something like this I guess:

"alternative": "Array.prototype.slice() or arr[0]"

Also, I think we should add more aliases here. For example _.head should be prevented when _.first is prevented, because they're basically the same.

Guide contributeur

Direction de recherche
Localisez le fichier lib/rules/rules.json à la ligne 51. Modifiez le champ alternative pour .first pour inclure 'arr[0]' en plus de Array.prototype.slice(). Ajoutez également des alias comme .head à empêcher lorsque .first est empêché. Vérifiez le modèle existant pour ajouter des alias dans le fichier de règles.
Stack technique
javascript
Domaine
documentation
Type d'issue
Fonctionnalité
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.
Moins d'une heure
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
basic JavaScriptJSON formatfamiliarity with lodash
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70