Skip to content

Updated README and add CI badges (open-telemetry#93) #249

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

Merged
merged 2 commits into from
Jun 17, 2022

Conversation

EddieEldridge
Copy link
Contributor

Description

Copy link
Contributor

@NathanielRN NathanielRN left a comment

Choose a reason for hiding this comment

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

This looks really useful! Thank you for doing it 🙂 Added some comments to improve formatting and understanding, please let me know what you think!

@EddieEldridge
Copy link
Contributor Author

Ok @NathanielRN, thanks for the feedback. I've included all of it and squashed the commit. It should be good now :)

Copy link
Contributor

@NathanielRN NathanielRN left a comment

Choose a reason for hiding this comment

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

I'll just batch these suggestions because they are typo fixes! Thanks again for your help :)

@NathanielRN NathanielRN merged commit 52268bf into open-telemetry:main Jun 17, 2022
alexis-hop pushed a commit to SafeHello/opentelemetry-lambda that referenced this pull request Jun 21, 2022
banderon1 pushed a commit to SafeHello/opentelemetry-lambda that referenced this pull request Jun 21, 2022
* Wrapper layer reduction from 11Mb to 9Mb removing unused instrumentation

* Updating wrapper layer dependencies

* Update nodejs/packages/layer/src/wrapper.ts

* Updated README and add CI badges (open-telemetry#249)

* [SAFE-467] buildkite pipeline (#1)

Collector and NodeJS layer pipeline

* Update .buildkite/pipeline.yml

Co-authored-by: Brian Anderson <[email protected]>
Co-authored-by: Medik <[email protected]>
alexis-hop added a commit to SafeHello/opentelemetry-lambda that referenced this pull request Jul 12, 2022
* Wrapper layer reduction from 11Mb to 9Mb removing unused instrumentation

* Updating wrapper layer dependencies

* Update nodejs/packages/layer/src/wrapper.ts

* Updated README and add CI badges (open-telemetry#249)

* [SAFE-467] buildkite pipeline (#1)

Collector and NodeJS layer pipeline

* Update .buildkite/pipeline.yml

Co-authored-by: Brian Anderson <[email protected]>
Co-authored-by: Medik <[email protected]>
alexis-hop added a commit to SafeHello/opentelemetry-lambda that referenced this pull request Jul 18, 2022
* Wrapper layer reduction from 11Mb to 9Mb removing unused instrumentation

* Updating wrapper layer dependencies

* Update nodejs/packages/layer/src/wrapper.ts

* Updated README and add CI badges (open-telemetry#249)

* [SAFE-467] buildkite pipeline (#1)

Collector and NodeJS layer pipeline

* Update .buildkite/pipeline.yml

Co-authored-by: Brian Anderson <[email protected]>
Co-authored-by: Medik <[email protected]>
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