graphql/graphql-js
Auf GitHub ansehenIs it possible to make findBreakingChange return line numbers?
Open
#1.341 geöffnet am 8. Mai 2018
7 Kommentare (7 Kommentare)1 Reaktion (1 Reaktion)0 zugewiesene Personen (0 zugewiesene Personen)TypeScript19.825 Stars (19.825 Stars)2.223 Forks (2.223 Forks)batch import
enhancementhelp wanted
Beschreibung
This would help to create some tools, like github pull request checks.
Contributor Guide
- Tech Stack
- typescript
- Domain
- backendapi
- Issue Type
- feature
- SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
- 4
- Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
- over 1 week
- AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
- stale
- KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
- needs investigation
- Voraussetzungen
- Knowledge of GraphQL schemaUnderstanding of graphql js source codeFamiliarity with AST and location tracking
- EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
- 20
- Research-Richtung
- 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.