graphql/graphql-js

Is it possible to make findBreakingChange return line numbers?

Ouverte

#1 341 ouverte le 8 mai 2018

Voir sur GitHubGuide contributeur
 (7 commentaires) (1 réaction) (0 personne assignée)TypeScript (2 223 forks)batch import
enhancementhelp wanted

Métriques du dépôt

Stars
 (19 825 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

This would help to create some tools, like github pull request checks.

Guide contributeur

Direction de recherche
Examinez l'implémentation actuelle de findBreakingChanges dans graphql js, probablement dans src/utilities/findBreakingChanges.ts. Comprenez comment les changements cassants sont détectés. Pour renvoyer les numéros de ligne, vous devriez modifier la fonction pour inclure les informations de localisation de l'AST GraphQL (par exemple, en utilisant la propriété 'loc' sur les nœuds). Étudiez comment d'autres parties de la base de code gèrent le suivi de localisation et envisagez d'étendre le type de retour pour inclure les numéros de ligne et de colonne. Consultez les tests existants pour findBreakingChanges pour garantir la compatibilité.
Stack technique
typescript
Domaine
backend
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.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitTypeScriptGraphQL
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60