Skip to content

ci(lint): lint the full tree, not just the diff#487

Open
krisarmstrong wants to merge 1 commit into
mainfrom
ci/full-tree-lint
Open

ci(lint): lint the full tree, not just the diff#487
krisarmstrong wants to merge 1 commit into
mainfrom
ci/full-tree-lint

Conversation

@krisarmstrong

Copy link
Copy Markdown
Collaborator

Summary

Drop --new-from-rev=HEAD~1 from the golangci-lint step so CI lints the whole tree instead of only changed lines. The pre-existing backlog that blocked this earlier is gone — a full-tree golangci-lint v2.12.2 run on origin/main reports 0 issues.

Linked Issue

Related to #486

Testing Evidence

golangci-lint run ./...   (full tree, origin/main, v2.12.2) -> 0 issues.

Security and Release Checklist

  • No secrets, tokens, credentials, or customer data are included.
  • Mutating routes, auth surfaces, permission checks, and output encoding were reviewed if touched.
  • Dependencies are pinned and justified if changed.
  • Documentation, screenshots, or operator notes were updated if behavior changed.

Backlog cleared and verified: golangci-lint v2.12.2 full-tree run on
origin/main reports 0 issues. Drop --new-from-rev=HEAD~1 so CI enforces
zero warnings across the whole repo instead of only changed lines.

Related to #486
@krisarmstrong krisarmstrong enabled auto-merge (squash) July 6, 2026 19:35
@github-actions github-actions Bot added the ci label Jul 6, 2026
@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

License Compliance Report

All dependencies pass license compliance checks

Go Dependencies

  • Unknown: 35 package(s)
  • MIT: 26 package(s)
  • BSD-3-Clause: 16 package(s)
  • Apache-2.0: 11 package(s)
  • BSD-2-Clause: 1 package(s)

npm Dependencies

See full report in workflow artifacts

Allowed Licenses: MIT, Apache-2.0, BSD-*, ISC, CC0-1.0, MPL-2.0
Forbidden: GPL, AGPL, SSPL (strong copyleft)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant