Add golangci-lint and stale workflows (#27)

Author: misraved

.github/workflows/golangci-lint.yml

@@ -0,0 +1,24 @@
+name: golangci-lint
+on:
+  push:
+    tags:
+      - v*
+    branches:
+      - main
+  pull_request:
+jobs:
+  golangci:
+    name: lint
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v3
+      - uses: actions/setup-go@v3
+        with:
+          go-version: '1.18.*'
+      - name: golangci-lint
+        uses: golangci/golangci-lint-action@v3
+        with:
+          # Optional: version of golangci-lint to use in form of v1.2 or v1.2.3 or `latest` to use the latest version
+          version: latest
+          args: --timeout=10m

.github/workflows/stale.yml

@@ -0,0 +1,42 @@
+name: "Stale Issues and PRs"
+on:
+  schedule:
+    - cron: "30 23 * * *"
+  workflow_dispatch:
+    inputs:
+      debugOnly:
+        description: "Set to true for a dry run"
+        required: false
+        default: "false"
+
+jobs:
+  stale:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Set environment variables
+        id: set-stale-action-env
+        run: |
+          echo "STALE_ISSUE_MESSAGE=This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 30 days." >> $GITHUB_ENV
+          echo "STALE_PR_MESSAGE=This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 30 days." >> $GITHUB_ENV
+          echo "CLOSE_ISSUE_MESSAGE=This issue was closed because it has been stalled for 90 days with no activity." >> $GITHUB_ENV
+          echo "CLOSE_PR_MESSAGE=This PR was closed because it has been stalled for 90 days with no activity." >> $GITHUB_ENV
+          echo "DAYS_BEFORE_STALE=60" >> $GITHUB_ENV
+          # DAYS_BEFORE_CLOSE is set to 30 because the we want to close the issue/PR after 90 days total (DAYS_BEFORE_STALE value is 60)
+          echo "DAYS_BEFORE_CLOSE=30" >> $GITHUB_ENV
+          echo "STALE_ISSUE_LABEL=stale" >> $GITHUB_ENV
+          echo "STALE_PR_LABEL=stale" >> $GITHUB_ENV
+
+      - name: Stale issues and PRs
+        id: stale-issues-or-prs
+        uses: actions/stale@v3
+        with:
+          repo-token: ${{ secrets.GITHUB_TOKEN }}
+          debug-only: ${{ github.event.inputs.debugOnly }}
+          stale-issue-message: ${{ env.STALE_ISSUE_MESSAGE }}
+          stale-pr-message: ${{ env.STALE_PR_MESSAGE }}
+          close-issue-message: ${{ env.CLOSE_ISSUE_MESSAGE }}
+          close-pr-message: ${{ env.CLOSE_PR_MESSAGE }}
+          days-before-stale: ${{ env.DAYS_BEFORE_STALE }}
+          days-before-close: ${{ env.DAYS_BEFORE_CLOSE }}
+          stale-issue-label: ${{ env.STALE_ISSUE_LABEL }}
+          stale-pr-label: ${{ env.STALE_PR_LABEL }}

zendesk/table_zendesk_organization.go

@@ -50,7 +50,7 @@ func listOrganization(ctx context.Context, d *plugin.QueryData, _ *plugin.Hydrat
 			PerPage: 100,
 		},
 	}
-	for true {
+	for {
 		organizations, page, err := conn.GetOrganizations(ctx, opts)
 		if err != nil {
 			return nil, err

zendesk/table_zendesk_search.go

@@ -48,7 +48,7 @@ func listSearch(ctx context.Context, d *plugin.QueryData, _ *plugin.HydrateData)
 		Query: q,
 	}
 	n := 0
-	for true {
+	for {
 		results, page, err := conn.Search(ctx, opts)
 		if err != nil {
 			return nil, err

zendesk/table_zendesk_ticket.go

@@ -81,7 +81,7 @@ func listTicket(ctx context.Context, d *plugin.QueryData, _ *plugin.HydrateData)
 			PerPage: 100,
 		},
 	}
-	for true {
+	for {
 		tickets, page, err := conn.GetTickets(ctx, opts)
 		if err != nil {
 			return nil, err

zendesk/table_zendesk_ticket_audit.go

@@ -45,7 +45,7 @@ func listTicketAudit(ctx context.Context, d *plugin.QueryData, _ *plugin.Hydrate
 		return nil, err
 	}
 	opts := zendesk.CursorOption{}
-	for true {
+	for {
 		ticketAudits, cursor, err := conn.GetAllTicketAudits(ctx, opts)
 		if err != nil {
 			return nil, err

zendesk/table_zendesk_trigger.go

@@ -55,7 +55,7 @@ func listTrigger(ctx context.Context, d *plugin.QueryData, _ *plugin.HydrateData
 			PerPage: 100,
 		},
 	}
-	for true {
+	for {
 		triggers, page, err := conn.GetTriggers(ctx, opts)
 		if err != nil {
 			return nil, err

zendesk/table_zendesk_user.go

@@ -80,7 +80,7 @@ func listUser(ctx context.Context, d *plugin.QueryData, _ *plugin.HydrateData) (
 			PerPage: 100,
 		},
 	}
-	for true {
+	for {
 		users, page, err := conn.GetUsers(ctx, opts)
 		if err != nil {
 			return nil, err

zendesk/utils.go

@@ -21,16 +21,14 @@ func connect(ctx context.Context, d *plugin.QueryData) (*zendesk.Client, error)
 	token := os.Getenv("ZENDESK_TOKEN")
 
 	zendeskConfig := GetConfig(d.Connection)
-	if &zendeskConfig != nil {
-		if zendeskConfig.SubDomain != nil {
-			subdomain = *zendeskConfig.SubDomain
-		}
-		if zendeskConfig.Email != nil {
-			user = *zendeskConfig.Email
-		}
-		if zendeskConfig.Token != nil {
-			token = *zendeskConfig.Token
-		}
+	if zendeskConfig.SubDomain != nil {
+		subdomain = *zendeskConfig.SubDomain
+	}
+	if zendeskConfig.Email != nil {
+		user = *zendeskConfig.Email
+	}
+	if zendeskConfig.Token != nil {
+		token = *zendeskConfig.Token
 	}
 
 	if subdomain == "" {
@@ -46,7 +44,10 @@ func connect(ctx context.Context, d *plugin.QueryData) (*zendesk.Client, error)
 	}
 
 	// example.zendesk.com
-	client.SetSubdomain(subdomain)
+	err = client.SetSubdomain(subdomain)
+	if err != nil {
+		return nil, err
+	}
 
 	// Authenticate with API token
 	client.SetCredential(zendesk.NewAPITokenCredential(user, token))