fix review comments and switch to spectral linter

This commit is contained in:
Michel Loiseleur 2024-11-29 15:14:32 +01:00
parent 36c6da990d
commit 0a1c52b1c5
Failed to extract signature
5 changed files with 12 additions and 9 deletions

View File

@ -29,14 +29,14 @@ jobs:
with:
go-version-file: go.mod
- name: Lint
- name: Lint go code
uses: golangci/golangci-lint-action@v6
with:
args: --timeout=30m
version: v1.60
shell: bash
- name: Run Vacuum
run: |
go run github.com/daveshanley/vacuum@latest lint -d api/webhook.yaml
shell: bash
# Run Spectral
- name: Lint OpenAPI spec
uses: stoplightio/spectral-action@2ad0b9302e32a77c1caccf474a9b2191a8060d83 # v0.8.11
with:
file_glob: 'api/*.yaml'

1
.spectral.yaml Normal file
View File

@ -0,0 +1 @@
extends: ["spectral:oas"]

View File

@ -60,8 +60,9 @@ licensecheck:
exit 1; \
fi
# Requires to install spectral. See https://github.com/stoplightio/spectral
oas-lint:
go run github.com/daveshanley/vacuum@latest lint -d api/webhook.yaml
spectral lint api/*.yaml
# Run all the linters
.PHONY: lint

View File

@ -1,7 +1,7 @@
---
openapi: "3.0.0"
info:
version: 0.14.0
version: v0.15.0
title: External DNS Webhook Server
description: >-
Implements the external DNS webhook endpoints.
@ -44,7 +44,7 @@ paths:
- example.com
'500':
description: |
Failed to provide the list of domains we serve.
Negociation failed.
/records:
get:

View File

@ -162,6 +162,7 @@ func TestAdjustEndpoints(t *testing.T) {
}
j, _ := json.Marshal(endpoints)
w.Write(j)
}))
defer svr.Close()