graphql/graphql-js
Voir sur GitHubIs it possible to make findBreakingChange return line numbers?
Open
#1 341 ouverte le 8 mai 2018
7 commentaires (7 commentaires)1 réaction (1 réaction)0 assignés (0 assignés)TypeScript19 825 stars (19 825 stars)2 223 forks (2 223 forks)batch import
enhancementhelp wanted
Description
This would help to create some tools, like github pull request checks.
Guide contributeur
- Stack technique
- typescript
- Domaine
- backendapi
- Type d'issue
- feature
- DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
- 4
- Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
- over 1 week
- Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
- stale
- ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
- needs investigation
- Prérequis
- Knowledge of GraphQL schemaUnderstanding of graphql js source codeFamiliarity with AST and location tracking
- Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
- 20
- Direction de recherche
- Examine the current implementation of findBreakingChanges in graphql js, likely in src/utilities/findBreakingChanges.ts. Understand how breaking changes are detected. To return line numbers, you would need to modify the function to include location information from the GraphQL AST (e.g., using the 'loc' property on nodes). Investigate how other parts of the codebase handle location tracking and consider extending the return type to include line and column numbers. Look at existing tests for findBreakingChanges to ensure compatibility.