Skip to content

refactor: cleanup GH gql query#100

Merged
OliverMKing merged 5 commits intoAzure:mainfrom
jsjoeio:jsjoeio/fix-gql-query
Oct 27, 2022
Merged

refactor: cleanup GH gql query#100
OliverMKing merged 5 commits intoAzure:mainfrom
jsjoeio:jsjoeio/fix-gql-query

Conversation

@jsjoeio
Copy link
Copy Markdown
Contributor

@jsjoeio jsjoeio commented Oct 25, 2022

This updates the graphql query used to fetch releases by:

  • returning only the first 10
  • ordering them in reverse by created_at

This is better because it does not rely on the default ordering by the GitHub graphql API.

Co-authored-by: Pieter Van der Haegen @vdhpieter

Testing Plan

I ran this query through the GitHub API explorer here:

image

This updates the graphql query used to fetch releases by:
- returning only the first 10
- ordering them in reverse by created_at

This is better because it does not rely on the default ordering by the
GitHub graphql API.

Co-authored-by: Pieter Van der Haegen <vdhpieter@outlook.com>
@jsjoeio jsjoeio marked this pull request as ready for review October 25, 2022 15:32
@jsjoeio jsjoeio requested a review from a team as a code owner October 25, 2022 15:32
OliverMKing
OliverMKing previously approved these changes Oct 25, 2022
@OliverMKing
Copy link
Copy Markdown
Contributor

LGTM. Thanks for the PR.

Co-authored-by: Sander Verweij <sverweij@users.noreply.github.com>
OliverMKing and others added 3 commits October 26, 2022 18:50
Co-authored-by: Sander Verweij <sverweij@users.noreply.github.com>
@OliverMKing
Copy link
Copy Markdown
Contributor

I merged the suggestions from @sverweij to get this out as soon as possible. Thank you for the work @jsjoeio. I'm going to get this in and released

Copy link
Copy Markdown
Collaborator

@davidgamero davidgamero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@jsjoeio
Copy link
Copy Markdown
Contributor Author

jsjoeio commented Oct 27, 2022

Team work makes the dream work!

@jsjoeio jsjoeio deleted the jsjoeio/fix-gql-query branch October 27, 2022 16:07
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.

5 participants