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.

Sign up for GitHub

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

Jump to bottom

Fix missing parenthesis in docker-deployment.md #3664

Merged
merged 2 commits into from
Dec 11, 2023
Merged

Fix missing parenthesis in docker-deployment.md #3664

merged 2 commits into from
Dec 11, 2023

Conversation

heitortsergent
Copy link
Contributor

@heitortsergent heitortsergent commented Dec 8, 2023 •
edited by chalin
Loading

Fixes a small typo and a missing parenthesis on the Docker deployment page.

https://deploy-preview-3664--opentelemetry.netlify.app/docs/demo/docker-deployment/#bring-your-own-backend

@heitortsergent heitortsergent requested review from a team December 8, 2023 22:16
@linux-foundation-easycla Linux Foundation: EasyCLA
Copy link

linux-foundation-easycla bot commented Dec 8, 2023 •
edited
Loading

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: heitortsergent / name: Heitor Tashiro Sergent (888383f)
  • ✅ login: svrnm / name: Severin Neumann (7c6e55a)

@svrnm
Copy link
Member

svrnm commented Dec 11, 2023

thank you @heitortsergent

@svrnm svrnm merged commit 74c06f6 into open-telemetry:main Dec 11, 2023
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mviitane mviitane mviitane approved these changes

@chalin chalin chalin approved these changes

@theletterf theletterf theletterf approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@heitortsergent @svrnm @theletterf @chalin @mviitane