sneak/prompts
1
0
Fork 0
Code Issues 11 Pull Requests Actions Packages Projects Releases Wiki Activity

style(go): add rule against type-only packages (per upaas #126 review) #2

Merged
sneak merged 2 commits from (deleted):add-no-type-only-packages-rule into main 2026-02-23 22:14:02 +01:00
Conversation 1 Commits 2 Files Changed 1
+23
clawbot commented 2026-02-23 20:48:14 +01:00
Collaborator
Copy Link
Copy Source

Adds a rule to CODE_STYLEGUIDE_GO.md: define types alongside their implementations, not in separate types, domain, or models packages.

Type-only packages cause alias imports and indicate poor package design.

Prompted by sneak's review comment on upaas PR #126.

Adds a rule to CODE_STYLEGUIDE_GO.md: define types alongside their implementations, not in separate `types`, `domain`, or `models` packages. Type-only packages cause alias imports and indicate poor package design. Prompted by sneak's review comment on upaas PR #126.
sneak was assigned by clawbot 2026-02-23 20:48:14 +01:00
clawbot added 1 commit 2026-02-23 20:48:14 +01:00
style(go): add rule against type-only packages f9dcef4c9e 
Types should live alongside their implementations, not in separate
'types', 'domain', or 'models' packages. Type-only packages cause
alias imports and indicate poor package design.

Prompted by review feedback on upaas PR #126.
clawbot added 1 commit 2026-02-23 20:56:18 +01:00
style(go): add Stringer rule for custom string-based types 7676ec16c3
clawbot commented 2026-02-23 20:56:23 +01:00
Author
Collaborator
Copy Link
Copy Source

Added the Stringer rule: custom string-based types must implement fmt.Stringer, and .String() should be used at SDK/library boundaries instead of string(v).

Added the Stringer rule: custom string-based types must implement `fmt.Stringer`, and `.String()` should be used at SDK/library boundaries instead of `string(v)`.
sneak merged commit dda0d01faa into main 2026-02-23 22:14:02 +01:00
sneak deleted branch add-no-type-only-packages-rule 2026-02-23 22:14:02 +01:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
merge-ready
PR approved and ready to merge
 needs-review
PR needs code review
 needs-rework
PR needs changes based on review
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
clawbot sneak (Jeffrey Paul)
No Assignees sneak
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/prompts#2