graphql/graphql-js
Ver no GitHubIs it possible to make findBreakingChange return line numbers?
Open
#1.341 aberto em 8 de mai. de 2018
7 comments (7 comments)1 reaction (1 reaction)0 assignees (0 assignees)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.
Guia do colaborador
- Pilha de tecnologia
- typescript
- Domain
- backendapi
- Tipo Issue
- feature
- DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
- 4
- Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
- over 1 week
- Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
- stale
- ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
- needs investigation
- Prerequisites
- Knowledge of GraphQL schemaUnderstanding of graphql js source codeFamiliarity with AST and location tracking
- Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
- 20
- Direção de pesquisa
- 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.