-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
[chore] Spelling receiver #37134
base: main
Are you sure you want to change the base?
[chore] Spelling receiver #37134
Conversation
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
…fication is on Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
…etplace Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
a0c9de0
to
7b60f5b
Compare
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.
Thanks for your work!
I wonder that why the misspell lint is not able to find those spelling errors?
@@ -1443,7 +1443,7 @@ type CommitComment implements Node & Comment & Deletable & Minimizable & Updatab | |||
viewerCanReact: Boolean! | |||
"Check if the current viewer can update this object." | |||
viewerCanUpdate: Boolean! | |||
"Reasons why the current viewer can not update this comment." | |||
"Reasons why the current viewer cannot update this comment." |
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.
Hm I believe this is a generated file, but there is no header on the file saying so. @adrielp can you confirm?
I think if the file is autogenerated then we don't want to modify it?
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.
I thought I dropped this somewhere...
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.
Checking everything is quite time consuming.
I checked the components I'm code owner for (and some more):
- elasticsearch
- github
- hostmetrics
- httpcheck
- nginx
- ntp
- osquery
- otelarrow
- podman
- pulsar
- sqlserver
- syslog
- udplog
- windowseventlog
- windowsperfcounter
- zipkin
If you're talking about https://pkg.go.dev/github.com/golangci/misspell:
check-spelling works against a full dictionary as opposed to just common misspellings. It's closer to cspell (but I'm currently working on one of the opentelemetry repositories that uses cspell, and I'm fixing a handful of typos there too...). |
Description
Fix spelling in receiver/
#37128 (review)
Link to tracking issue