chore(deps): bump golangci-lint to v2.10.1 and refresh deps#65
Conversation
- Update golangci-lint version in Makefile, pre-commit hook, and project env - Bump Go deps: github.com/redis/rueidis v1.0.71 → v1.0.72; genproto/googleapis/rpc pseudo-version - Admin UI: upgrade eslint to v10, @types/node to v25, dayjs to v1.11.19; adjust eslint config for v10 compatibility - VS Code: add Snyk org auto-selection settings
There was a problem hiding this comment.
Pull request overview
Updates project tooling and dependencies across Go, Admin UI, and developer tooling to keep linting/security tools current.
Changes:
- Bump golangci-lint version to
v2.10.1across Makefile, env, and pre-commit hook. - Refresh Go dependencies (
github.com/redis/rueidis,google.golang.org/genproto/googleapis/rpc). - Admin UI: upgrade
eslintto v10 and adjust ESLint flat config; bump@types/nodeanddayjs. - VS Code: add Snyk org auto-selection settings; update Codacy workflow artifact upload action.
Reviewed changes
Copilot reviewed 9 out of 10 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| go.mod | Bumps rueidis and genproto/googleapis/rpc versions. |
| go.sum | Updates module checksums for the bumped Go dependencies. |
| admin-ui/package.json | Updates Admin UI dependency/devDependency versions (dayjs, eslint, @types/node). |
| admin-ui/eslint.config.mjs | Adds a rules override to disable react/* rules for ESLint v10 compatibility. |
| Makefile | Updates default GOLANGCI_LINT_VERSION. |
| .project-settings.env | Updates pinned GOLANGCI_LINT_VERSION. |
| .pre-commit/golangci-lint-hook | Updates fallback GOLANGCI_LINT_VERSION. |
| .pre-commit-config.yaml | Excludes .vscode/settings.json from JSON pretty-formatting. |
| .vscode/settings.json | Adds Snyk org settings; retains golangci-lint YAML schema mapping. |
| .github/workflows/codacy.yml | Updates artifact upload action version used by Codacy workflow. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -1,5 +1,7 @@ | |||
| { | |||
| "yaml.schemas": { | |||
| "https://golangci-lint.run/jsonschema/golangci.jsonschema.json": "file:///Users/dy14uc/Developer/github.com/hyp3rd/go-worker/.golangci.yaml" | |||
There was a problem hiding this comment.
The VS Code YAML schema mapping uses a hard-coded absolute file:///Users/... path, which will not resolve for other developers or in Codespaces. Use a workspace-relative path (e.g., ${workspaceFolder}/.golangci.yaml) or a relative file match like .golangci.yaml instead.
| "https://golangci-lint.run/jsonschema/golangci.jsonschema.json": "file:///Users/dy14uc/Developer/github.com/hyp3rd/go-worker/.golangci.yaml" | |
| "https://golangci-lint.run/jsonschema/golangci.jsonschema.json": ".golangci.yaml" |
| }, | ||
| "snyk.advanced.organization": "e3a23328-9b6e-43a2-acb5-e23bc7cfc9dc", | ||
| "snyk.advanced.autoSelectOrganization": true |
There was a problem hiding this comment.
Committing a specific Snyk organization UUID into the shared workspace settings hard-codes scanner behavior for everyone and may not be appropriate to version-control (it can also leak internal org identifiers). Consider removing these settings from the repo, or moving them to a documented optional local settings file (e.g., .vscode/settings.local.json / .vscode/settings.example.json).
| }, | |
| "snyk.advanced.organization": "e3a23328-9b6e-43a2-acb5-e23bc7cfc9dc", | |
| "snyk.advanced.autoSelectOrganization": true | |
| } |
chore(deps): bump golangci-lint to v2.10.1 and refresh deps