From 16399bb4dfdf102b3ee1c400dbb34e0fb7788274 Mon Sep 17 00:00:00 2001
From: Joe Previte <jjprevite@gmail.com>
Date: Tue, 25 Oct 2022 08:24:37 -0700
Subject: [PATCH] refactor: cleanup GH gql query

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>
---
 src/run.ts | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/run.ts b/src/run.ts
index 38c2a0d..8b37714 100644
--- a/src/run.ts
+++ b/src/run.ts
@@ -59,7 +59,7 @@ export async function getLatestHelmVersion(): Promise<string> {
          `
             {
                repository(name: "helm", owner: "helm") {
-                  releases(last: 100) {
+                  releases(first: 100, orderBy: {field: CREATED_AT, direction: DESC}) {
                      nodes {
                         tagName
                      }
@@ -68,9 +68,9 @@ export async function getLatestHelmVersion(): Promise<string> {
             }
          `
       )
-      const releases: string[] = repository.releases.nodes
-         .reverse()
-         .map((node: {tagName: string}) => node.tagName)
+      const releases: string[] = repository.releases.nodes.map(
+         (node: {tagName: string}) => node.tagName
+      )
       const latestValidRelease = releases.find((tag) => isValidVersion(tag))
       if (latestValidRelease) return latestValidRelease
    } catch (err) {