Skip to content

docs: fix duplicated word in keyvalue.go comment#8336

Open
quyentonndbs wants to merge 1 commit into
open-telemetry:mainfrom
quyentonndbs:docs/fix-duplicated-is-in-keyvalue-comment
Open

docs: fix duplicated word in keyvalue.go comment#8336
quyentonndbs wants to merge 1 commit into
open-telemetry:mainfrom
quyentonndbs:docs/fix-duplicated-is-in-keyvalue-comment

Conversation

@quyentonndbs
Copy link
Copy Markdown

Fix a duplicated word in a comment in log/keyvalue.go.

Line 315 read:
// first if a slog.Kind is is not handled. It is

The word "is" appears twice consecutively. This removes the duplicate.

@linux-foundation-easycla
Copy link
Copy Markdown

CLA Missing ID

  • ❌ The email address for the commit (a9bdb54) is not linked to the GitHub account, preventing the EasyCLA check. Consult this Help Article and GitHub Help to resolve. (To view the commit's email address, add .patch at the end of this PR page's URL.) For further assistance with EasyCLA, please visit our EasyCLA portal and chat with our support bot.

@Dipanshusinghh
Copy link
Copy Markdown
Contributor

I can take a look at this if nobody is already working on additional review/testing. I’ll verify the change locally and check if there are any related comment/docs cleanups nearby as well.

Copy link
Copy Markdown
Contributor

@Dipanshusinghh Dipanshusinghh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed the change locally — the duplicate word removal looks correct and I don’t see any impact outside the comment itself.
Only thing pending right now seems to be the CLA check.

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