`EvaluationResult` and related structs' fields are public

Aberta
  • Algo errado?
#45 0 comentários 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
3/5
Tempo estimado
Meio dia
Facilidade para iniciantes
52/100
Tipo de issue
Refatoração
Clareza
Razoavelmente clara
Status de atividade
Ativa
Stack de tecnologia
rust
Domínio
backend

Direção de pesquisa

Start by finding the EvaluationResult, FlagResult and SegmentResult struct definitions and their public field declarations, then check which of these are only read by callers. Decide which structs are read-only before changing anything, since the issue says "etc." without listing them all. Done means those fields are private, public getters exist, and the crate's existing tests still pass.

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

Descrição

Declaring EvaluationResult, FlagResult, SegmentResult etc. fields as public suggests that we support consumers constructing them, which we in fact do not.

For read-only structs, we should make the fields private, and expose public getters instead.

Linguagem predominante
Rust
Estrelas
2
Forks
0
Merge médio
23h 26min
PRs com merge (30d)
3

Preparar o ambiente

Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de Flagsmith/flagsmith-rust-flag-engine

Todas as issues de Flagsmith/flagsmith-rust-flag-engine

Issues semelhantes

Mais issues de Rust

Receba novas issues na sua caixa de entrada

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