From dd95edf1cd249dbc62c98a775aeb80f0cc0928f4 Mon Sep 17 00:00:00 2001
From: Nick Rundquist <nrundquist@atlassian.com>
Date: Thu, 14 Mar 2019 15:34:19 -0500
Subject: [PATCH] Remove `q:` style queries as an option for pipelines
 endpoints.

---
 scripts/generate-routes.js | 16 ++++++++++------
 src/routes/routes.json     | 32 --------------------------------
 2 files changed, 10 insertions(+), 38 deletions(-)

diff --git a/scripts/generate-routes.js b/scripts/generate-routes.js
index 9c628c0..2f8e974 100644
--- a/scripts/generate-routes.js
+++ b/scripts/generate-routes.js
@@ -181,15 +181,19 @@ const addFilterAndSortParams = routesObject => {
       let isPaginatedList = /^list/i.test(apiName)
       let returnsPaginated =
         apiObject.returns && /^paginated/i.test(apiObject.returns)
+      let isPipelines = /pipelines/i.test(apiObject.url)
 
       if (isPaginatedList || returnsPaginated) {
+        parameters = [
+          { in: 'query', name: 'page', require: false, type: 'string' },
+          { in: 'query', name: 'pagelen', required: false, type: 'integer' },
+          { in: 'query', name: 'sort', required: false, type: 'string' }
+        ]
+        if (!isPipelines) {
+          parameters.push({ in: 'query', name: 'q', required: false, type: 'string' })
+        }
         setParameters(apiObject, {
-          parameters: [
-            { in: 'query', name: 'page', require: false, type: 'string' },
-            { in: 'query', name: 'pagelen', required: false, type: 'integer' },
-            { in: 'query', name: 'q', required: false, type: 'string' },
-            { in: 'query', name: 'sort', required: false, type: 'string' }
-          ]
+          parameters: parameters
         })
       }
     })
diff --git a/src/routes/routes.json b/src/routes/routes.json
index 878ad32..2a3643a 100644
--- a/src/routes/routes.json
+++ b/src/routes/routes.json
@@ -3382,10 +3382,6 @@
           "in": "query",
           "type": "integer"
         },
-        "q": {
-          "in": "query",
-          "type": "string"
-        },
         "repo_slug": {
           "in": "path",
           "required": true,
@@ -3415,10 +3411,6 @@
           "in": "query",
           "type": "integer"
         },
-        "q": {
-          "in": "query",
-          "type": "string"
-        },
         "repo_slug": {
           "in": "path",
           "required": true,
@@ -3448,10 +3440,6 @@
           "in": "query",
           "type": "integer"
         },
-        "q": {
-          "in": "query",
-          "type": "string"
-        },
         "repo_slug": {
           "in": "path",
           "required": true,
@@ -3481,10 +3469,6 @@
           "in": "query",
           "type": "integer"
         },
-        "q": {
-          "in": "query",
-          "type": "string"
-        },
         "repo_slug": {
           "in": "path",
           "required": true,
@@ -3519,10 +3503,6 @@
           "required": true,
           "type": "string"
         },
-        "q": {
-          "in": "query",
-          "type": "string"
-        },
         "repo_slug": {
           "in": "path",
           "required": true,
@@ -3552,10 +3532,6 @@
           "in": "query",
           "type": "integer"
         },
-        "q": {
-          "in": "query",
-          "type": "string"
-        },
         "repo_slug": {
           "in": "path",
           "required": true,
@@ -5575,10 +5551,6 @@
           "in": "query",
           "type": "integer"
         },
-        "q": {
-          "in": "query",
-          "type": "string"
-        },
         "sort": {
           "in": "query",
           "type": "string"
@@ -6074,10 +6046,6 @@
           "in": "query",
           "type": "integer"
         },
-        "q": {
-          "in": "query",
-          "type": "string"
-        },
         "sort": {
           "in": "query",
           "type": "string"