APPSRE-6327 fragment dependency is unique

app-sre / qenerate

Code Generator for GraphQL Query and Fragment Data Classes

Apache License 2.0

9 stars 7 forks source link

APPSRE-6327 fragment dependency is unique #51

Closed fishi0x01 closed 2 years ago

fishi0x01 commented 2 years ago

A query can have multiple references to the same fragment. Currently each reference leads to an additional dependency, also when it is the same fragment.

This PR uses a set instead of a list, to ensure fragment dependencies are only added once.

fishi0x01 commented 2 years ago

Issue occurred here first: https://github.com/app-sre/qontract-reconcile/pull/2811