kmesh-net/kmesh

Make protobuf generated api file under same package with .proto

Aberta

#1.024 aberto em 8 de nov. de 2024

Ver no GitHubGuia do colaborador
 (4 comentários) (0 reação) (1 responsável)Go (239 forks)auto 404
good first issuekind/enhancement

Métricas do repositório

Stars
 (739 estrelas)
Métricas de merge de PR
 (Mesclagem média 6d 4h) (8 fundiu PRs em 30d)

Description

What would you like to be added:

Currently we make the https://github.com/kmesh-net/kmesh/tree/main/api/workloadapi generated files into https://github.com/kmesh-net/kmesh/tree/main/api/v2/workloadapi. It does work well from fdunctionality, in my local dev IDE, goland/vscode, the code navigation does not work well, when i want to naviagate to https://github.com/kmesh-net/kmesh/blob/52d8f9ef7de5641e2718d1e25b641ba416e2c65c/api/v2/workloadapi/workload.pb.go#L707, but it jump to https://github.com/kmesh-net/kmesh/blob/52d8f9ef7de5641e2718d1e25b641ba416e2c65c/api/workloadapi/workload.proto#L138

Guia do colaborador

Direção de pesquisa
Investigue a configuração de geração de código protobuf (buf.gen.yaml ou invocação protoc) para garantir que os arquivos Go gerados sejam colocados no mesmo diretório do pacote que os arquivos .proto, ajustando caminhos de saída e mapeamentos de importação.
Pilha de tecnologia
go
Domain
developer experience
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitGoprotobuf
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75