sneak/prompts
1
0
Çatalla 0
Kod Konular Değişiklik İstekleri İşlemler Paketler Projeler Sürüm Wiki Aktivite

styleguide: add rule against type-only packages #1

Kapalı
clawbot (silindi): add-no-types-package-rule içindeki 1 işlemeyi main ile birleştirmek istiyor
şuradan çek: (silindi): add-no-types-package-rule
birleştir: sneak:main
Sohbet 2 İşleme 1 Değiştirilen Dosyalar 1 +8
clawbot 2026-02-23 20:47:53 +01:00 yorum yaptı
Katkıcı
Bağlantıyı Kopyala

Adds a rule to the Go styleguide: do not create packages just to hold types (e.g. types, domain). Types should be defined alongside the implementations that use them.

This came up during review of sneak/upaas PR #126 where an internal/domain package existed solely to hold type definitions.

The new rule is placed before the existing "define struct types near constructors" rule, as they are thematically related.

Adds a rule to the Go styleguide: do not create packages just to hold types (e.g. `types`, `domain`). Types should be defined alongside the implementations that use them. This came up during review of sneak/upaas PR #126 where an `internal/domain` package existed solely to hold type definitions. The new rule is placed before the existing "define struct types near constructors" rule, as they are thematically related.
sneak 2026-02-23 20:47:53 +01:00 clawbot tarafından atandı
clawbot 1 işlemeyi 2026-02-23 20:47:54 +01:00 ekledi
styleguide: add rule against type-only packages 97f5d8a91f 
Types should be defined alongside the implementations that use them,
not in separate 'types' or 'domain' packages. This avoids awkward
import aliases and keeps types close to their usage.
clawbot 2026-02-23 20:49:33 +01:00 değişiklik isteğini kapattı
sneak 2026-02-23 20:54:46 +01:00 yorum yaptı
Sahibi
Bağlantıyı Kopyala

@clawbot why was this closed?

@clawbot why was this closed?
clawbot 2026-02-23 20:55:06 +01:00 yorum yaptı
Yazar
Katkıcı
Bağlantıyı Kopyala

Sorry — two sub-agents raced and created duplicate PRs. This one got closed in favor of PR #2 which covers the same change. I'll make sure #2 also includes the Stringer rule (not just the no-type-only-packages rule).

Sorry — two sub-agents raced and created duplicate PRs. This one got closed in favor of PR #2 which covers the same change. I'll make sure #2 also includes the Stringer rule (not just the no-type-only-packages rule).

Değişiklik isteği kapatıldı

Dal silindiğinden bu değişiklik isteği yeniden açılamaz.
Bu konuşmaya katılmak için oturum aç.
Gözden Geçirenler
Gözden geçiren yok
Etiketler
Etiketleri temizle
merge-ready
PR approved and ready to merge
 needs-review
PR needs code review
 needs-rework
PR needs changes based on review
Etiket Yok
Kilometre Taşı
Öge yok
Kilometre Taşı Yok
Projeler
Projeleri temizle
Proje yok
Atananlar
Atamaları Temizle
clawbot sneak
Atanan Kişi Yok sneak
2 Katılımcı
Bildirimler
Bitiş Tarihi
Bitiş tarihi atanmadı.
Bağımlılıklar

Bağımlılık yok.

Referans: sneak/prompts#1
"(silindi): add-no-types-package-rule" Dalını Sil

Bir dalı silmek kalıcıdır. Her ne kadar silinen dal tamamen kaldırılana kadar çok kısa bir süre yaşamını sürdürse de, çoğu durumda bu işlem GERİ ALINAMAZ. Devam edilsin mi?