Skip to content

Commit generated GraphQL schema - #4721

Merged
patrick91 merged 5 commits into
mainfrom
codex/graphql-schema-check
Aug 13, 2026
Merged

Commit generated GraphQL schema#4721
patrick91 merged 5 commits into
mainfrom
codex/graphql-schema-check

Conversation

@patrick91

@patrick91 patrick91 commented Aug 12, 2026

Copy link
Copy Markdown
Member

No description provided.

@vercel

vercel Bot commented Aug 12, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
pycon Error Error Aug 13, 2026 7:28am

@claude

claude Bot commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Adds a CI workflow that regenerates the GraphQL schema and diffs it against a committed backend/schema.graphql, and commits that initial schema file.

Issue: the new check fails on this PR as-is

The "Generated schema is up to date" check run for this PR's own head commit (ca1b4e1) fails:

-union Block = NewsGridSection | CommunitiesSection | SliderCardsSection | SponsorsSection | SchedulePreviewSection | TextSection | DynamicContentDisplaySection | SpecialGuestSection | LiveStreamingSection | InformationSection | CheckoutSection | SocialsSection | KeynotersSection | HomepageHero | HomeIntroSection | CMSMap
+union Block = SpecialGuestSection | HomepageHero | SponsorsSection | DynamicContentDisplaySection | LiveStreamingSection | InformationSection | KeynotersSection | SocialsSection | TextSection | CommunitiesSection | SliderCardsSection | CheckoutSection | SchedulePreviewSection | CMSMap | NewsGridSection | HomeIntroSection

The Block union member ordering isn't deterministic between runs (same schema, different member order each time manage.py graphql_schema runs), so git diff --exit-code flags a spurious diff. As committed, this check will flake on essentially every PR regardless of whether the schema actually changed, which defeats the purpose of adding it as a required check. This needs to be fixed (e.g. sort union members before printing, or otherwise make the Block union's member order deterministic) before this is merge-worthy — otherwise CI will be red on this PR and on unrelated future PRs.

@codecov

codecov Bot commented Aug 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.67%. Comparing base (5a977f9) to head (ca1b4e1).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4721   +/-   ##
=======================================
  Coverage   92.67%   92.67%           
=======================================
  Files         360      360           
  Lines       11184    11184           
  Branches      896      896           
=======================================
  Hits        10365    10365           
  Misses        696      696           
  Partials      123      123           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@patrick91
patrick91 marked this pull request as ready for review August 13, 2026 07:24
@patrick91
patrick91 force-pushed the codex/graphql-schema-check branch from 7313a7c to ca1b4e1 Compare August 13, 2026 07:24
@patrick91
patrick91 merged commit 729c949 into main Aug 13, 2026
7 of 9 checks passed
@patrick91
patrick91 deleted the codex/graphql-schema-check branch August 13, 2026 07:29
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.

1 participant