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

example http server #56

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

example http server #56

wants to merge 5 commits into from

Conversation

kakkun61
Copy link
Collaborator

@kakkun61 kakkun61 commented Mar 1, 2023

This example uses:

  • hs-opentelemetry-api
  • hs-opentelemetry-sdk
  • hs-opentelemetry-propagator-datadog
  • hs-opentelemetry-propagator-w3c
  • hs-opentelemetry-instrumentation-wai
  • hs-opentelemetry-instrumentation-http-client

@kakkun61 kakkun61 force-pushed the example-http-server branch 2 times, most recently from f6f396d to 75cdfa2 Compare March 3, 2023 07:48
@kakkun61 kakkun61 mentioned this pull request Apr 21, 2023
Merged
@kakkun61 kakkun61 self-assigned this Aug 18, 2023
kakkun61 and others added 3 commits August 18, 2023 19:20
@kakkun61
Show log
02c1ac3
@kakkun61
format
ed83495
@kakkun61
http server example
2e6cf69 
This example uses:

- hs-opentelemetry-api
- hs-opentelemetry-sdk
- hs-opentelemetry-propagator-datadog
- hs-opentelemetry-propagator-w3c
- hs-opentelemetry-instrumentation-wai
- hs-opentelemetry-instrumentation-http-client
iand675
iand675 requested changes Aug 22, 2023
View reviewed changes
examples/http-server/main.hs Outdated
@@ -0,0 +1,54 @@
{-# LANGUAGE OverloadedStrings #-}
Copy link
Owner

Choose a reason for hiding this comment

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

Small nitpick: main.hs should probably be Main.hs instead.

hie.yaml Outdated Show resolved Hide resolved
kakkun61 and others added 2 commits August 23, 2023 01:56
@kakkun61 @iand675
Update hie.yaml
8990628 
Co-authored-by: Ian Duncan <ian@iankduncan.com>
@kakkun61
main.hs → Main.hs
f47d815
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@iand675 iand675 iand675 requested changes

@parsonsmatt parsonsmatt Awaiting requested review from parsonsmatt

Requested changes must be addressed to merge this pull request.

Assignees

@kakkun61 kakkun61

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@kakkun61 @iand675