• Joined on 2026-02-08
clawbot commented on pull request sneak/upaas#152 2026-03-01 20:35:43 +01:00
Split Dockerfile into lint + build stages for faster CI feedback

Review: PASS

Reviewing issue #151 — split Dockerfile into lint + build stages for faster CI feedback (post-rework).

Checklist

clawbot commented on pull request sneak/upaas#152 2026-03-01 20:31:16 +01:00
Split Dockerfile into lint + build stages for faster CI feedback

Rework complete — removed unnecessary goimports install from the lint stage.

What changed: Removed the RUN go install golang.org/x/tools/cmd/goimports@... line from the lint stage…

clawbot pushed to feature/split-dockerfile-lint-build-stages at sneak/upaas 2026-03-01 20:31:01 +01:00
d661c81955 Remove unnecessary goimports install from lint stage
clawbot commented on pull request sneak/upaas#152 2026-03-01 19:38:34 +01:00
Split Dockerfile into lint + build stages for faster CI feedback

Code Review — PASS

Reviewed PR #152 which closes issue #151.

Checklist

  • **golangci-li…
clawbot commented on pull request sneak/upaas#152 2026-03-01 19:33:27 +01:00
Split Dockerfile into lint + build stages for faster CI feedback

Work Summary

Split the Dockerfile into separate lint and build stages per issue #151.

Lint stage uses golangci/golangci-lint:v2.10.1

clawbot created pull request sneak/upaas#152 2026-03-01 19:32:49 +01:00
Split Dockerfile into lint + build stages for faster CI feedback
clawbot pushed to feature/split-dockerfile-lint-build-stages at sneak/upaas 2026-03-01 19:32:38 +01:00
2c4c1beaf9 split Dockerfile into lint + build stages for faster CI feedback
clawbot created branch feature/split-dockerfile-lint-build-stages in sneak/upaas 2026-03-01 19:32:37 +01:00
clawbot opened issue sneak/upaas#151 2026-03-01 19:27:09 +01:00
Split Dockerfile into lint + build stages for faster CI feedback
clawbot commented on pull request sneak/dnswatcher#41 2026-03-01 16:57:10 +01:00
REPO_POLICIES compliance audit

Closing as duplicate — PR #40 already covers issue #39 and is merge-ready.

clawbot closed pull request sneak/dnswatcher#41 2026-03-01 16:57:09 +01:00
REPO_POLICIES compliance audit
clawbot created pull request sneak/dnswatcher#41 2026-03-01 16:56:29 +01:00
REPO_POLICIES compliance audit
clawbot pushed to fix/issue-39-repo-policies at sneak/dnswatcher 2026-03-01 16:56:16 +01:00
4dede6af9d update README: reference REPO_POLICIES.md, add License/Author sections, document new Makefile targets
b84c2f3b34 remove CLAUDE.md and CONVENTIONS.md, replaced by REPO_POLICIES.md
a65a14b8ec add fmt-check, hooks, docker targets; add 30s test timeout
b7b9629bc3 add missing files: LICENSE, REPO_POLICIES.md, .editorconfig, .dockerignore
Compare 4 commits »
clawbot created branch fix/issue-39-repo-policies in sneak/dnswatcher 2026-03-01 16:56:16 +01:00
clawbot commented on pull request sneak/dnswatcher#40 2026-03-01 16:54:56 +01:00
REPO_POLICIES compliance audit

Review: PASS

Reviewed all changes in PR #40 (closing issue #39).

Checklist

clawbot commented on pull request sneak/dnswatcher#40 2026-03-01 16:54:33 +01:00
REPO_POLICIES compliance audit

Review: PASS

clawbot commented on pull request sneak/dnswatcher#28 2026-03-01 16:52:04 +01:00
fix: 700ms query timeout, proper iterative resolution (closes #24)

Code Review: PASS

Reviewer: clawbot (fresh review per sneak request)

Summary

PR correctly fixes the root cause of hanging make check: resolveARecord and resolveNSRecursive

clawbot created pull request sneak/dnswatcher#40 2026-03-01 16:51:58 +01:00
REPO_POLICIES compliance audit
clawbot created branch repo-policies-compliance in sneak/dnswatcher 2026-03-01 16:51:46 +01:00
clawbot pushed to repo-policies-compliance at sneak/dnswatcher 2026-03-01 16:51:46 +01:00
c7e5c03239 Update README for REPO_POLICIES compliance
b6f3ed314e Remove CLAUDE.md and CONVENTIONS.md
da312f5c7b Add fmt-check, hooks, docker targets and test timeout to Makefile
80c6236cf1 Add REPO_POLICIES.md, .editorconfig, and .dockerignore
Compare 4 commits »