Skip to content

Conversation

@erikwrede
Copy link
Member

@erikwrede erikwrede commented Oct 24, 2025

When sorting a schema lexicographically, the enum values lost their extensions.

@codspeed-hq
Copy link

codspeed-hq bot commented Oct 24, 2025

CodSpeed Performance Report

Merging #245 will not alter performance

Comparing erikwrede:fix-keep-enum-value-extensions-on-lex-sorting (72e6445) with main (4b0ada2)

Summary

✅ 14 untouched

@Cito
Copy link
Member

Cito commented Oct 24, 2025

Thank you @erikwrede

@Cito Cito merged commit 431b0f5 into graphql-python:main Oct 24, 2025
14 checks passed
@Cito Cito changed the title fix: don't loose extensions when sorting enum values lexicographically. fix: don't loose extensions of enum values when sorting schemas Oct 24, 2025
@Cito
Copy link
Member

Cito commented Oct 24, 2025

Actually input values and the schema itself had the same problem, fixed that already, and backported it as well.

@erikwrede
Copy link
Member Author

@Cito still meant to check for that, great you found it first :) thanks for the quick merge

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants