Azure / setup-helm

Github Action for installing Helm
MIT License
141 stars 48 forks source link

refactor: cleanup GH gql query #100

Closed jsjoeio closed 2 years ago

jsjoeio commented 2 years ago

This updates the graphql query used to fetch releases by:

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
OliverMKing commented 2 years ago

LGTM. Thanks for the PR.

OliverMKing commented 2 years ago

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

jsjoeio commented 2 years ago

Team work makes the dream work!