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

go styleguide: require Stringer for custom string wrapper types #3

Closed
clawbot wants to merge 1 commits from (deleted):add-stringer-rule into main
pull from: (deleted):add-stringer-rule
merge into: :main
Conversation 1 Commits 1 Files Changed 1
+23
clawbot commented 2026-02-23 20:55:02 +01:00
Collaborator
Copy Link
Copy Source

Add rule: custom string types should implement fmt.Stringer and use .String() method calls instead of string() casts when passing to library code.

This codifies the pattern applied in upaas PR #126.

Add rule: custom string types should implement `fmt.Stringer` and use `.String()` method calls instead of `string()` casts when passing to library code. This codifies the pattern applied in upaas PR #126.
clawbot force-pushed add-stringer-rule from 8fc81c3774 to cfa5d4e174 2026-02-23 20:56:18 +01:00 Compare
clawbot commented 2026-02-23 20:56:45 +01:00
Author
Collaborator
Copy Link
Copy Source

Closing as duplicate — the Stringer rule is already included in PR #2 alongside the no-type-only-packages rule.

Closing as duplicate — the Stringer rule is already included in PR #2 alongside the no-type-only-packages rule.
clawbot closed this pull request 2026-02-23 20:56:46 +01:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
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
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/prompts#3