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

trivial: Fix golint errors #495

Merged
merged 2 commits into from
Aug 7, 2023
Merged

trivial: Fix golint errors #495

merged 2 commits into from
Aug 7, 2023

Conversation

tetromino
Copy link
Collaborator

Google's linter is complaining about the following:

  • comment on exported method Int.Cmp should be of the form "Cmp ..."
  • receiver name x should be consistent with previous receiver name i for Int
  • exported method Float.Cmp should have comment or be unexported
  • receiver name x should be consistent with previous receiver name f for Float
  • method parameter y_ should be y

Specifically, the following:

* comment on exported method Int.Cmp should be of the form "Cmp ..."
* receiver name x should be consistent with previous receiver name i for Int
* exported method Float.Cmp should have comment or be unexported
* receiver name x should be consistent with previous receiver name f for Float
* method parameter y_ should be y
@tetromino tetromino requested a review from adonovan August 1, 2023 19:15
starlark/int.go Outdated Show resolved Hide resolved
starlark/value.go Outdated Show resolved Hide resolved
@tetromino tetromino merged commit 2aa7575 into google:master Aug 7, 2023
7 checks passed
@tetromino tetromino deleted the lint branch August 7, 2023 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants