Skip to content
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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Package name as exported variable/type/function prefix removal #1181

Closed
Adirio opened this issue Nov 11, 2019 · 3 comments · Fixed by #1291
Closed

Package name as exported variable/type/function prefix removal #1181

Adirio opened this issue Nov 11, 2019 · 3 comments · Fixed by #1291
Assignees
Labels
kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. kind/feature Categorizes issue or PR as related to a new feature.

Comments

@Adirio
Copy link
Contributor

Adirio commented Nov 11, 2019

Prefixing an exported variable/type/function with the package name is redundant as they will be called as package.PackageType and should be avoided.

Umbrella issue: #1176

/kind cleanup

@Adirio Adirio added the kind/feature Categorizes issue or PR as related to a new feature. label Nov 11, 2019
@k8s-ci-robot k8s-ci-robot added the kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. label Nov 11, 2019
@Adirio
Copy link
Contributor Author

Adirio commented Nov 13, 2019

Requires #1186 and #1190 to be merged as they modify the same lines:

  • pkg/scaffold/v1/crd/crd_sample.go:39,51,55
  • pkg/scaffold/v2/prometheus/monitor.go:15,23
  • pkg/scaffold/v2/webhook/webhook.go:57,84,89

@camilamacedo86
Copy link
Member

HI, could you please assign the issue to you in order to avoid other dev get it since you gave a PR already one for this one? Also, devs will usually filter for the unassigned ones to know what they can do to contribute with.

PS.: To do it just add a new comment with /assign @your user

@Adirio
Copy link
Contributor Author

Adirio commented Nov 14, 2019

/assign

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants