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
When reading the examples, I propose to streamline function names to correspond to the naming scheme of fmt, log and testing: fmt.Printf(), log.Printf(), t.Errorf() are all using lowercase f, whereas your package uses WarningF()DebugF() with capital Fs.
Proposal: Use lowercase f in your package as well, e.g. Warningf()
The text was updated successfully, but these errors were encountered:
Your project looks really nice!
When reading the examples, I propose to streamline function names to correspond to the naming scheme of
fmt
,log
andtesting
:fmt.Printf()
,log.Printf()
,t.Errorf()
are all using lowercasef
, whereas your package usesWarningF()
DebugF()
with capitalF
s.Proposal: Use lowercase
f
in your package as well, e.g.Warningf()
The text was updated successfully, but these errors were encountered: