[PR #14] [MERGED] Add common linters #30

New issue

Closed

opened 2026-02-26 18:43:17 +03:00 by kerem · 0 comments

kerem commented

2026-02-26 18:43:17 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/devnullvoid/pvetui/pull/14
Author: @devnullvoid
Created: 7/3/2025
Status: ✅ Merged
Merged: 7/3/2025
Merged by: @devnullvoid

Base: develop ← Head: codex/refactor-project-for-go-best-practices

📝 Commits (5)

4b602fc ✨ Add Windows ARM64 build support
f0276b7 Add common linters
445e9f1 Merge branch 'develop' into codex/refactor-project-for-go-best-practices
f47554c Fix lint config output format
c5e259c Merge branch 'codex/refactor-project-for-go-best-practices' of github.com:devnullvoid/proxmox-tui into codex/refactor-project-for-go-best-practices

📊 Changes

14 files changed (+163 additions, -104 deletions)

View changed files

📝 .github/workflows/release.yml (+3 -1)
📝 .golangci.yml (+8 -81)
➕ AGENTS.md (+19 -0)
📝 CHANGELOG.md (+8 -1)
📝 Makefile (+1 -0)
📝 internal/cache/cache_test.go (+2 -2)
📝 internal/ssh/client.go (+42 -9)
➕ internal/ssh/client_test.go (+34 -0)
➕ internal/ssh/executor.go (+21 -0)
📝 internal/ui/app_runner.go (+1 -0)
📝 pkg/api/auth_test.go (+2 -1)
📝 test/integration/api_client_integration_test.go (+4 -1)
📝 test/integration/cache_integration_test.go (+16 -6)
📝 test/integration/end_to_end_integration_test.go (+2 -2)

📄 Description

Summary

expand GolangCI configuration with a core set of linters
document linter update in the changelog

Testing

make build
make lint
go vet ./...
make test

https://chatgpt.com/codex/tasks/task_e_68649b45b1308328b97af3ebc450347a

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/devnullvoid/pvetui/pull/14 **Author:** [@devnullvoid](https://github.com/devnullvoid) **Created:** 7/3/2025 **Status:** ✅ Merged **Merged:** 7/3/2025 **Merged by:** [@devnullvoid](https://github.com/devnullvoid) **Base:** `develop` ← **Head:** `codex/refactor-project-for-go-best-practices` --- ### 📝 Commits (5) - [`4b602fc`](https://github.com/devnullvoid/pvetui/commit/4b602fc7ce58cba321ece46b594ae490582fbf9e) ✨ Add Windows ARM64 build support - [`f0276b7`](https://github.com/devnullvoid/pvetui/commit/f0276b79bab3baa7b450ac64abd1d142ecffef9b) Add common linters - [`445e9f1`](https://github.com/devnullvoid/pvetui/commit/445e9f141582cd6fb01c603a30188dc194ba1837) Merge branch 'develop' into codex/refactor-project-for-go-best-practices - [`f47554c`](https://github.com/devnullvoid/pvetui/commit/f47554ca3ebcb5638a5ef4ae496439b6b7f90374) Fix lint config output format - [`c5e259c`](https://github.com/devnullvoid/pvetui/commit/c5e259ca240d54fdf95a24a8bd38fb7c52f4a9e1) Merge branch 'codex/refactor-project-for-go-best-practices' of github.com:devnullvoid/proxmox-tui into codex/refactor-project-for-go-best-practices ### 📊 Changes **14 files changed** (+163 additions, -104 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/release.yml` (+3 -1) 📝 `.golangci.yml` (+8 -81) ➕ `AGENTS.md` (+19 -0) 📝 `CHANGELOG.md` (+8 -1) 📝 `Makefile` (+1 -0) 📝 `internal/cache/cache_test.go` (+2 -2) 📝 `internal/ssh/client.go` (+42 -9) ➕ `internal/ssh/client_test.go` (+34 -0) ➕ `internal/ssh/executor.go` (+21 -0) 📝 `internal/ui/app_runner.go` (+1 -0) 📝 `pkg/api/auth_test.go` (+2 -1) 📝 `test/integration/api_client_integration_test.go` (+4 -1) 📝 `test/integration/cache_integration_test.go` (+16 -6) 📝 `test/integration/end_to_end_integration_test.go` (+2 -2) </details> ### 📄 Description ## Summary - expand GolangCI configuration with a core set of linters - document linter update in the changelog ## Testing - `make build` - `make lint` - `go vet ./...` - `make test` ------ https://chatgpt.com/codex/tasks/task_e_68649b45b1308328b97af3ebc450347a --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>