Khan/genqlient

Be less strict about selection-set ordering

Open

#93 ouverte le 15 sept. 2021

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)Go (141 forks)github user discovery
enhancementgood first issuehelp wanted

Métriques du dépôt

Stars
 (1 310 stars)
Métriques de merge PR
 (Métriques PR en attente)

Description

For both the bindings option, and the soon-to-be-added typename option, we validate that two selection sets are equal. Right now we demand that they even have the fields in the same order; ideally we'd allow any equivalent selections. That's surprisingly much work, although it's quite well separated from the rest of the codebase (just modify selectionsMatch in generate/validation.go).

Guide contributeur

Direction de recherche
Modifiez la fonction `selectionsMatch` dans `generate/validation.go` pour comparer les ensembles de sélection indépendamment de l'ordre des champs. Envisagez une comparaison basée sur des ensembles plutôt que sur l'ordre séquentiel. Testez avec des sélections équivalentes dans divers ordres.
Stack technique
go
Domaine
tooling
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
Go
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60