`EvaluationResult` and related structs' fields are public
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- Une demi-journée
- Accessibilité débutants
- 52/100
Piste de recherche
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.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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.
- Langage dominant
- Rust
- Étoiles
- 2
- Forks
- 0
- Merge moyen
- 23 h 26 min
- PR mergées (30 j)
- 3
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de Flagsmith/flagsmith-rust-flag-engine
-
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
-
Dependency DashboardOuverte
Difficulté 4/5 3-5 jours Accessibilité débutants 25/100
-
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
Toutes les issues de Flagsmith/flagsmith-rust-flag-engine
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
pop-os/cosmic-settings#2296 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
jonathanong/no-mistakes#3431 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
Eyevinn/open-live-ingest#36 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Windows YAML bind mounts fail path validation while equivalent --mount-dir CLI mounts succeedOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
superradcompany/microsandbox#2071 ·
Les mainteneurs répondent en général sous 1 jour