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
// Foo does the magicfuncFoo() {}
// Bar is blah blah//// Deprecated: use FoofuncBar() {}
//nolint:whateverfuncBaz() {}
// Deprecated: is deprecatedfuncQux() {}
funcQuux() {}
And is now reported as: "exported func Baz should have comment or be unexported"
The last one is reported as: "exported func Quux should have comment or be unexported"
The issue is with Qux, the error reported is "comment should be like Qux ...", but in fact here it should be "exported function should have a comment", the very same way Baz is reported
The text was updated successfully, but these errors were encountered:
ccoVeille
changed the title
exported:
exported: comment starting with "Deprecated:” should be ignored when checking if a comment is present
Feb 12, 2025
nit: in the next PR please add a comment explaining that .Text removes directives from the comment block Originally posted by @chavacava in #1209 (comment)
The first two are valid
The third one is fixed by a previous PR:
And is now reported as: "exported func Baz should have comment or be unexported"
The last one is reported as: "exported func Quux should have comment or be unexported"
The issue is with
Qux
, the error reported is "comment should be like Qux ...", but in fact here it should be "exported function should have a comment", the very same wayBaz
is reportedThe text was updated successfully, but these errors were encountered: