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

Sync main #2

Merged
merged 4 commits into from
Dec 1, 2022
Merged

Sync main #2

merged 4 commits into from
Dec 1, 2022

Conversation

Temppus
Copy link
Owner

@Temppus Temppus commented Dec 1, 2022

Fixes #.

Changes

Please provide a brief description of the changes here.

For significant contributions please make sure you have completed the following items:

  • Appropriate CHANGELOG.md updated for non-trivial changes
  • Design discussion issue #
  • Changes in public API reviewed

pellared and others added 4 commits November 28, 2022 15:54
@pellared
Update README.md (#3951)
6c73e36
@vishweshbankwar @TimothyMothra @CodeBlanch
[SqlClient] Clarify behavior when using System.Data.SqlClient with …
e376aef 
….NET Framework (#3937)

* Add System.Data.SqlClient clarification for netfmk

* wrap comment

* readme update

* readme

* Update src/OpenTelemetry.Instrumentation.SqlClient/README.md

Co-authored-by: Timothy Mothra <tilee@microsoft.com>

* resolve PR comments

Co-authored-by: Timothy Mothra <tilee@microsoft.com>
Co-authored-by: Mikel Blanchard <mblanchard@macrosssoftware.com>
@hgaol @reyang
[DOC] Update Jaeger tutorial to use OTLP Exporter (#3940)
2604607 
* [DOC] Update Jaeger tutorial to use OTLP Exporter

* fix HttpClient could not be found in .NET framework error during CI

* fix markdown lint issue

* Update jaeger example .csproj file

* Update docs/trace/getting-started-jaeger/getting-started-jaeger.csproj

* Remove redundant options setting and update README

Co-authored-by: Reiley Yang <reyang@microsoft.com>
@moonbox3
Fix dotnet run jaeger command. Include comment on how to access local…
2116966 
…host Jaeger UI. (#3957)
@Temppus Temppus merged commit 7d98a5d into Temppus:main Dec 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
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
@Temppus @pellared @vishweshbankwar @hgaol @moonbox3