From 9a0b853c4e9e4d81eed112caf89d366729ef1d09 Mon Sep 17 00:00:00 2001
From: Kordian Bruck <kordianbruck@users.noreply.github.com>
Date: Sun, 22 Sep 2024 20:34:26 +0200
Subject: [PATCH] Configure lint in actions, add PR template and dependency
 updates through renovate (#187)

* Configure renovate for keeping dependencies updated

* Add a PR template.

* Configure lint CI and refresh the build/test steps.

* Swap renovate for gh native dependabot instead as its easier to activate.

* Ignore existing issues and annotate the code in the PR diff for better visibility.

* Improve name for the main workflow.
---
 .github/dependabot.yml                        | 15 ++++++++++
 .github/pull_request_template.md              | 12 ++++++++
 .../workflows/{go.yml => go-build-test.yml}   | 13 ++++-----
 .github/workflows/golangci-lint.yml           | 29 +++++++++++++++++++
 4 files changed, 61 insertions(+), 8 deletions(-)
 create mode 100644 .github/dependabot.yml
 create mode 100644 .github/pull_request_template.md
 rename .github/workflows/{go.yml => go-build-test.yml} (77%)
 create mode 100644 .github/workflows/golangci-lint.yml

diff --git a/.github/dependabot.yml b/.github/dependabot.yml
new file mode 100644
index 0000000..ad58f45
--- /dev/null
+++ b/.github/dependabot.yml
@@ -0,0 +1,15 @@
+version: 2
+# See https://docs.github.com/en/github/administering-a-repository/configuration-options-for-dependency-updates for options
+updates:
+  - package-ecosystem: "gomod"
+    directory: "/" # Location of package manifests
+    schedule:
+      interval: "monthly"
+  - package-ecosystem: "docker"
+    directory: "/" # Location of package manifests
+    schedule:
+      interval: "monthly"
+  - package-ecosystem: "github-actions"
+    directory: "/"
+    schedule:
+      interval: "monthly"
\ No newline at end of file
diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md
new file mode 100644
index 0000000..fb7ef2d
--- /dev/null
+++ b/.github/pull_request_template.md
@@ -0,0 +1,12 @@
+### Motivation and Context
+<!-- Why is this change required? What problem does it solve? -->
+<!-- If it fixes an open issue, please link to the issue here. -->
+
+### Description
+<!-- Describe your changes in detail, what does your code do? How does it do it? -->
+
+### Steps for Testing
+<!-- Please describe in detail how a reviewer can test your changes. -->
+
+### Screenshots
+<!-- Add screenshots to demonstrate the changes. -->
\ No newline at end of file
diff --git a/.github/workflows/go.yml b/.github/workflows/go-build-test.yml
similarity index 77%
rename from .github/workflows/go.yml
rename to .github/workflows/go-build-test.yml
index 03233ed..566599f 100644
--- a/.github/workflows/go.yml
+++ b/.github/workflows/go-build-test.yml
@@ -1,26 +1,23 @@
-name: Go
+name: Go Build and Test
 
 on:
   push:
-    branches: [ master ]
+    branches:
+      - main
+      - master
   pull_request:
-    branches: [ master ]
 
 jobs:
-
   build:
     runs-on: ubuntu-latest
     steps:
     - uses: actions/checkout@v4
-
     - name: Set up Go
       uses: actions/setup-go@v5
       with:
         cache-dependency-path: './go.sum'
         go-version-file: './go.mod'
-
     - name: Build
       run: go build -v ./...
-
     - name: Test
-      run: go test -v ./...
+      run: go test -v ./...
\ No newline at end of file
diff --git a/.github/workflows/golangci-lint.yml b/.github/workflows/golangci-lint.yml
new file mode 100644
index 0000000..00fbab3
--- /dev/null
+++ b/.github/workflows/golangci-lint.yml
@@ -0,0 +1,29 @@
+name: golangci-lint
+on:
+  push:
+    branches:
+      - main
+      - master
+  pull_request:
+
+permissions:
+  contents: read
+  # Optional: allow read access to pull request. Use with `only-new-issues` option.
+  pull-requests: read
+  # Optional: allow write access to checks to allow the action to annotate code in the PR.
+  checks: write
+
+jobs:
+  golangci:
+    name: lint
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-go@v5
+        with:
+          go-version: stable
+      - name: golangci-lint
+        uses: golangci/golangci-lint-action@v6
+        with:
+          version: v1.60
+          only-new-issues: true
\ No newline at end of file