Skip to content

Comments

chore(deps): bump golangci-lint to v2.10.1 and refresh deps#65

Merged
hyp3rd merged 2 commits intomainfrom
feat/admin-ui
Feb 21, 2026
Merged

chore(deps): bump golangci-lint to v2.10.1 and refresh deps#65
hyp3rd merged 2 commits intomainfrom
feat/admin-ui

Conversation

@hyp3rd
Copy link
Owner

@hyp3rd hyp3rd commented Feb 21, 2026

chore(deps): bump golangci-lint to v2.10.1 and refresh deps

  • 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

- 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
Copilot AI review requested due to automatic review settings February 21, 2026 20:23
@hyp3rd hyp3rd merged commit b74b43f into main Feb 21, 2026
2 checks passed
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.1 across Makefile, env, and pre-commit hook.
  • Refresh Go dependencies (github.com/redis/rueidis, google.golang.org/genproto/googleapis/rpc).
  • Admin UI: upgrade eslint to v10 and adjust ESLint flat config; bump @types/node and dayjs.
  • 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"
Copy link

Copilot AI Feb 21, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Suggested change
"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"

Copilot uses AI. Check for mistakes.
Comment on lines +4 to +6
},
"snyk.advanced.organization": "e3a23328-9b6e-43a2-acb5-e23bc7cfc9dc",
"snyk.advanced.autoSelectOrganization": true
Copy link

Copilot AI Feb 21, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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).

Suggested change
},
"snyk.advanced.organization": "e3a23328-9b6e-43a2-acb5-e23bc7cfc9dc",
"snyk.advanced.autoSelectOrganization": true
}

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant