Skip to content

github: Move to a typed graphql client instead of handrolled queries#251

Closed
jerry-skydio wants to merge 1 commit into
jerry/revup/main/fakeforgefrom
jerry/revup/main/githubtyped
Closed

github: Move to a typed graphql client instead of handrolled queries#251
jerry-skydio wants to merge 1 commit into
jerry/revup/main/fakeforgefrom
jerry/revup/main/githubtyped

Conversation

@jerry-skydio

Copy link
Copy Markdown
Collaborator

Download and checkin github's schema, then create a codegen client with
ariadne-codegen. This significantly improves the readability of the
github forge code.

Download and checkin github's schema, then create a codegen client with
ariadne-codegen. This significantly improves the readability of the
github forge code.
@jerry-skydio

Copy link
Copy Markdown
Collaborator Author

Reviews in this chain:
#249 forge: Genericize github handling into a forge abstraction
 └#250 test: Expand upload tests with fake_forge
  └#251 github: Move to a typed graphql client instead of handrolled queries

@jerry-skydio

jerry-skydio commented May 20, 2026

Copy link
Copy Markdown
Collaborator Author
# head base diff date summary
0 becb9eec 48d65d5d diff May 20 12:37 PM 41 files changed, 66813 insertions(+), 614 deletions(-)
1 975373bf 48d65d5d diff May 20 12:43 PM 3 files changed, 59 insertions(+), 8 deletions(-)
2 1ff126dc 48d65d5d diff May 20 12:47 PM 4 files changed, 27 insertions(+), 53 deletions(-)
3 7a5e78b4 48d65d5d diff May 20 13:06 PM 3 files changed, 723 insertions(+), 422 deletions(-)

@jerry-skydio jerry-skydio force-pushed the jerry/revup/main/githubtyped branch 3 times, most recently from 1ff126d to 7a5e78b Compare May 20, 2026 20:06
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