Update lint config

[tangrufus]

1. golangci-lint: Lint with integration & wporg_live build tags
2. Run gofmt via golangci-lint
3. Remove standalone go vet
go vet is ran by golangci-lint. Thus, it does not need to be ran separately.

[Copilot]

Pull request overview

This PR updates the project’s linting setup to consolidate formatting/vet under golangci-lint and to lint code compiled with the integration and wporg_live build tags, aligning local linting and CI behavior.

Changes:

• Configure golangci-lint to run with integration and wporg_live build tags and enable gofmt via golangci-lint.
• Simplify CI by removing standalone gofmt and go vet jobs (relying on golangci-lint instead).
• Update integration tests to explicitly ignore resp.Body.Close() errors to satisfy errcheck.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
internal/integration/smoke_test.go	Adjusts resp.Body.Close() defers in integration smoke tests.
internal/integration/helpers_test.go	Adjusts resp.Body.Close() defer in shared integration test helper.
Makefile	Removes standalone gofmt and go vet from make lint.
.golangci.yml	Adds build tags for linting and enables gofmt via golangci-lint config.
.github/workflows/ci.yml	Removes separate formatting and vet jobs, keeping lint + tidy + tests.

Comments suppressed due to low confidence (1)

internal/integration/smoke_test.go:183

• io.ReadAll(resp.Body)'s error is being intentionally ignored (body, _ := ...). With errcheck enabled and integration now included in golangci-lint build tags, this will be reported as an unchecked error and fail lint. Please handle the error (and fail the test on read failure) instead of discarding it.

		defer func() { _ = resp.Body.Close() }()
		if resp.StatusCode != 200 {
			t.Errorf("package detail status: got %d, want 200", resp.StatusCode)
		}
		body, _ := io.ReadAll(resp.Body)

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Adding integration/wporg_live to run.build-tags will cause golangci-lint to lint the integration test packages too. There are existing errcheck violations under internal/integration (e.g., unchecked io.ReadAll errors), so this config change will likely make the lint job fail until those are fixed or excluded.

Suggested change

	build-tags:
	- integration
	- wporg_live