99designs / 99designs/gqlgen

Use embed for all graphql files sources

Aberta
#2,198 4 comentários 0 reações 0 responsáveis Ver no GitHub
Linguagem predominante
Go
Estrelas
10.8k
Forks
1.3k
Merge médio
2d 36min
PRs com merge (30d)
26

Descrição

@vikstrous Now that I have tried to solve https://github.com/99designs/gqlgen/issues/2196 by myself I have realized that you are only using embed for the first file in the list of possible graphql files of the config `schema` field.

Can you try adding another in your project (maybe even using the glob way) and check that it's not just me?

Like my one:

```yml
schema:
- schema/*.graphql
- ../../pkg/**/gqlgen/schema/*.graphql
```

Just as an example, this is an exceprt of the generated.go file after your improvement:

```go
var sources = []*ast.Source{
{Name: "schema/schema.graphql", Input: sourceData("schema/schema.graphql"), BuiltIn: false},
{Name: "../../pkg/player/gqlgen/schema/player.graphql", Input: `type Player {
id: ID!
createdAt: Time!
name: String!
age: Int!
}
`, BuiltIn: false},
// and so on many other sources here...
```

Guia de contribuição

Abrir o guia de contribuição

Direção de pesquisa

Examine a lógica de geração de código que processa o campo schema na configuração do gqlgen, provavelmente em cmd ou internal/codegen. A issue mostra que apenas o primeiro padrão glob usa embed; você precisa estender isso para todos os padrões. Comece encontrando onde as fontes são construídas a partir das entradas do schema e, em seguida, execute a geração em um projeto de teste com vários padrões glob para verificar o fix.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
go
Domínio
build-system
Tipo de issue
Bug
Dificuldade
3/5
Tempo estimado
1-2 dias
Status de atividade
Estagnada
Clareza
Claramente especificada
Facilidade para iniciantes
55/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.