You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Major docs/development.md carries forward inaccurate repo-local facts into the new canonical guide. make test is described as running “race detection and coverage,” but the Makefile’s test target runs go test -v -race ./...; coverage is under test-cover. The repo structure also lists root main.go, but the binary entrypoint is cmd/slck/main.go and the Makefile builds ./cmd/slck. Fix both before merging so the repo-local guide is trustworthy.
Major docs/development.md still has one stale quick-command fact: make install is documented as installing to $GOPATH/bin, but the Makefile’s install target runs install -m 755 bin/$(BINARY) /usr/local/bin/. Update that comment so the new repo-local guide reflects the actual install destination.
The updated diff resolves the local factual issues. The PR now matches the initiative pattern: peer thin entrypoints, Slack-specific facts in docs/development.md, canonical source/local references for shared standards and automation, and no duplicated CI/release mechanics in the touched guidance.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Verification
Closes #181