Conversation
…1246) - Move all .go sources from repo root into /go/ subtree (canonical layout) - Add go.work at repo root + external/go submodule (https://github.com/dappcore/go.git@dev) - Migrate to v0.9.0 contracts: core.Result, core.E, c.Fs(), c.Process(), c.I18n() - Replace banned stdlib imports (os/os.exec/fmt/errors/strings/path/log/json/bytes) - Add docs skeleton (AGENTS.md) - Backfill Test<File>_<Symbol>_{Good,Bad,Ugly} triplets + Example<Symbol> coverage Verification: - GOWORK=off go build ./... clean - GOWORK=off go vet ./... clean - GOWORK=off go test -count=1 -short ./... clean - audit.sh verdict: COMPLIANT (every counter at 0) Co-authored-by: Codex <noreply@openai.com> Closes tasks.lthn.sh/view.php?id=1246
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (16)
📝 WalkthroughWalkthroughThe PR refactors test naming conventions across multiple files from descriptive suffixes ( Changes
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 golangci-lint (2.11.4)Error: can't load config: unsupported version of the configuration: "" See https://golangci-lint.run/docs/product/migration-guide for migration instructions Review rate limit: 3/5 reviews remaining, refill in 20 minutes and 50 seconds. Comment |
|


Restructure go-ratelimit to canonical /go/ subtree layout, full v0.9.0 contract migration, audit verdict COMPLIANT (197 baseline findings → 0). Closes Mantis #1246.
Summary by CodeRabbit
Release Notes
New Features
Documentation
Tests
Chores