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

Fix typos in "Signals" documentation section #3825

Merged
merged 2 commits into from
Jan 20, 2024

Conversation

fabrideci
Copy link
Contributor

This PR includes the code required to fix two simple typos in the "Logs" and "Traces" pages of the "Concept - Signals" documentation section.

@fabrideci fabrideci requested a review from a team January 19, 2024 14:27
Copy link

linux-foundation-easycla bot commented Jan 19, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: fabrideci / name: Fabrizio De Cicco (4b0f276)
  • ✅ login: cartermp / name: Phillip Carter (6545137)

Copy link
Contributor

@chalin chalin left a comment

Choose a reason for hiding this comment

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

Thanks for the fixes @fabrideci. We'll need you to sign the CLA before we can merge this.

@@ -298,7 +298,7 @@ Each span has a status. The three possible values are:
The default value is `Unset`. A span status that is `Unset` means that the
operation it tracked successfully completed without an error.

When a span status is `Error`, then that means some error ocurred in the
Copy link
Contributor

Choose a reason for hiding this comment

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

@svrnm - we should find out why cSpell doesn't detect this. I made some quick tests but wasn't able to find out why. Maybe a bug with the its dictionary?

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

@cartermp cartermp merged commit 6e3bf58 into open-telemetry:main Jan 20, 2024
14 checks passed
jaydeluca pushed a commit to jaydeluca/opentelemetry.io that referenced this pull request Jan 31, 2024
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.

5 participants