-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
馃尡 Enforce godoc comments for exported functions #5817
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thx a lot!
A few typos + make -C test/infrastructure/docker generate-manifests
test/infrastructure/docker/exp/api/v1alpha3/dockermachinepool_types.go
Outdated
Show resolved
Hide resolved
test/infrastructure/docker/exp/api/v1alpha4/dockermachinepool_types.go
Outdated
Show resolved
Hide resolved
test/infrastructure/docker/exp/api/v1beta1/dockermachinepool_types.go
Outdated
Show resolved
Hide resolved
This updates a filter in our golangci-lint configuration to start enforcing the presence of godoc comments on any functions or variables that are exported. This aligns with most of the rest of the codebase and makes it so there is (somewhat) useful documentation for anything that can be accessed externally from a package. Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
8a79c79
to
a0890c8
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: fabriziopandini The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Thanks for improving or code quality! |
Absolutely agree! Thx a lot. Things like this are minor issues, but they raise the quality overall and the linter will help us to keep it there :) /lgtm |
What this PR does / why we need it:
This updates a filter in our golangci-lint configuration to start
enforcing the presence of godoc comments on any functions or variables
that are exported in the
infrastructure/docker
path.This aligns with most of the rest of the codebase and
makes it so there is (somewhat) useful documentation for anything that
can be accessed externally from a package.